Repository navigation
Conversation
Deploying liney with
|
| Latest commit: |
d20b87d
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://1fa5b91e.liney.pages.dev |
| Branch Preview URL: | https://fix-qr-login-rpc-lifecycle.liney.pages.dev |
|
Additional QR-expiry audit: inspected the pinned upstream LINEJS v3.5.0 implementation at https://github.com/evex-dev/linejs/blob/v3.5.0/packages/linejs/base/login/mod.ts. |
CI update — 2026-11-10Latest PR head:
Artifact: liney-electron-linux-preview Scope note: CI smoke tests run without a LINE account. They verify app/worker boot and packaging, not actual QR authentication, token restore, or message delivery. Live QA with a separate test LINE account is still required before merge. |
Summary
Fix the QR login RPC lifecycle independently from PR #20.
auth.qr.startas soon as LINEJS emits the QR URL or PIN challenge, instead of waiting for the full authentication promise.auth.readyremains the success signal.auth.errorwhen authentication fails after the startup RPC has already been acknowledged, and display that error in the UI.Verification
QR expiry / lock lifecycle audit
Inspected the exact pinned upstream LINEJS v3.5.0 implementation:
longPollingMaxCountandlongPollingIntervalSec; the implementation retries timed-out polls only up to that count, then throws. It is not an unbounded wait under the normal request-timeout path.Live QA is still outstanding. CI does not prove authentication with a real LINE account. Test only with a separate test account. This PR remains Draft until that test is completed.
Related: #17. Separate from transport-status PR #20.