fix webhook payment processing and server URL logging

- Remove 'processing' intermediate status (not in payments ENUM) — fetch
pending payment directly then mark completed in one step
- Fix double-protocol in Portal base log (PORTAL_HOST already has scheme)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

src/routes/webhooks.routes.js

@@ -53,29 +53,23 @@ async function handlePaymentCompleted(data) {
   if (!reference) return;
 
   // ── WiFi guest purchase ────────────────────────────────────────────────────
-  // Atomically claim the payment — only one concurrent webhook can advance it
-  const claim = await db.query(
-    `UPDATE payments SET status = 'processing' WHERE reference = ? AND status = 'pending'`,
+  const payment = await db.query(
+    `SELECT p.*, wp.duration_seconds,
+            wp.down_limit, wp.down_unit, wp.up_limit, wp.up_unit, wp.name AS plan_name,
+            d.name AS device_name, c.phone AS client_phone
+     FROM payments p
+     JOIN wifi_plans wp ON wp.id = p.plan_id
+     JOIN devices d ON d.id = p.device_id
+     JOIN clients c ON c.id = p.client_id
+     WHERE p.reference = ? AND p.status = 'pending' LIMIT 1`,
     [reference]
-  );
-
-  if (claim.affectedRows === 1) {
-    const payment = await db.query(
-      `SELECT p.*, wp.duration_seconds,
-              wp.down_limit, wp.down_unit, wp.up_limit, wp.up_unit, wp.name AS plan_name,
-              d.name AS device_name, c.phone AS client_phone
-       FROM payments p
-       JOIN wifi_plans wp ON wp.id = p.plan_id
-       JOIN devices d ON d.id = p.device_id
-       JOIN clients c ON c.id = p.client_id
-       WHERE p.reference = ? LIMIT 1`,
-      [reference]
-    ).then(r => r[0]);
+  ).then(r => r[0]);
 
+  if (payment) {
     const code = generateCode();
 
     // Create access token
-    const tokenResult = await db.query(
+    await db.query(
       `INSERT INTO access_tokens
          (client_id, device_id, plan_id, payment_id, code, duration_seconds,
           down_limit, down_unit, up_limit, up_unit, status)
@@ -87,7 +81,7 @@ async function handlePaymentCompleted(data) {
       ]
     );
 
-    // Mark payment completed (trigger fires here — credits client balance)
+    // Mark payment completed — trigger fires here and credits client balance
     await db.query(
       `UPDATE payments SET
          status = 'completed',

src/server.js

@@ -30,7 +30,8 @@ async function start() {
   // 3. Listen
   app.listen(PORT, () => {
     console.log(`[Server] Listening on http://0.0.0.0:${PORT}`);
-    console.log(`[Server] Portal base: ${process.env.PORTAL_SCHEME}://${process.env.PORTAL_HOST}`);
+    const host = (process.env.PORTAL_HOST || '').replace(/^https?:\/\//, '');
+    console.log(`[Server] Portal base: ${process.env.PORTAL_SCHEME || 'http'}://${host}`);
   });
 }