Skip to content

Fix: add retry logic to WhatsApp QR authentication network error#144

Closed
rahim86 wants to merge 1 commit intoqwibitai:mainfrom
rahim86:main
Closed

Fix: add retry logic to WhatsApp QR authentication network error#144
rahim86 wants to merge 1 commit intoqwibitai:mainfrom
rahim86:main

Conversation

@rahim86
Copy link
Copy Markdown

@rahim86 rahim86 commented Feb 8, 2026

Problem Statement:

  • Users unable to complete QR authentication successfully

    Error Message while authenticating WhatsApp
    {"level":50,"time":1770559453555,"pid":29069,"hostname":"mac.lan","fullErrorNode":{"tag":"stream:error","attrs":{"code":"515"}},"msg":"stream errored out"}

    Connection failed. Please try again.

Solution:

  • Introduce retry counts and retry delays

Testing:

  • QR code authentication now completes successfully
  • Proper error handling and console messages
  • Handles temporary disconnects during handshake

Authored-By: Abdul Rahim Mohammed Abubacker

Type of Change

  • Fix - bug fix or security fix to source code

@rahim86 rahim86 requested a review from gavrielc as a code owner February 8, 2026 14:36
@TomGranot
Copy link
Copy Markdown
Collaborator

Changes needed:

  • Good retry logic for WhatsApp QR auth (MAX_RETRIES=5, RETRY_DELAY_MS=3000).
  • The diff targets src/whatsapp-auth.ts which is correct.
  • Please rebase to confirm clean apply. Note that feat: add WhatsApp auth retry #63 also adds retry logic to the same file (for 515 restarts specifically) — these may need to be coordinated to avoid conflicts.

@rahim86
Copy link
Copy Markdown
Author

rahim86 commented Feb 14, 2026

@TomGranot Rebase the branch as requested.Please review

Problem:
- Users unable to complete QR authentication successfully

    Error Message while authenticating WhatsApp
        {"level":50,"time":1770559453555,"pid":29069,"hostname":"mac.lan","fullErrorNode":{"tag":"stream:error","attrs":{"code":"515"}},"msg":"stream errored out"}

    Connection failed. Please try again.

Solution:
- Introduce retry counts and retry delays

Testing:
- QR code authentication now completes successfully
- Proper error handling and console messages
- Handles temporary disconnects during handshake

Authored-By: Abdul Rahim Mohammed Abubacker
@TomGranot
Copy link
Copy Markdown
Collaborator

Closing in favor of PR #63 which implements the same WhatsApp auth retry logic for stream error 515 with a cleaner approach. Both fix the same issue — consolidating to #63. Thank you for the contribution!

@TomGranot TomGranot closed this Feb 19, 2026
eplisov added a commit to eplisov/nanoclaw that referenced this pull request Apr 6, 2026
PostgreSQL port exposure and default password are now handled by OneCLI
itself (≥ April 2026). Steps 1-2 changed from "fix" to "verify".
iptables rules for app ports 10254/10255 remain critical — OneCLI does
not address this.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
kenansun-dev-bot bot pushed a commit to kenansun-dev/nanoclaw-github-copilot that referenced this pull request Apr 12, 2026
…tai#144)

1. Tunnel URL: get from 'devtunnel show' instead of hardcoding asse region
2. .env: strip old Teams section before appending (no duplicates on re-run)

1 file.

Co-authored-by: Kenan VM Claw <vm-claw@kenan.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants