Skip to content

Commit 72ced2f

Browse files
ampagentIsaac Miller
andcommitted
Record post-approval error-boundary regression fix
Signed-off-by: Amp <amp@ampcode.com> Amp-Thread-ID: https://ampcode.com/threads/T-01a0c095-4917-7617-9dac-e99f1ef22d41 Co-authored-by: Isaac Miller <isaac@cmpnd.ai>
1 parent c9d45be commit 72ced2f

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

‎DOWNSTREAM_PATCHES.md‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,12 @@ unpatched and do not rewrite published integration branches.
5353
all three Meta routes with remint/endpoint change/logout, cooperative and
5454
non-cooperative cancelled login workers, and valid/rejected verification
5555
with and without concurrent replacement.
56+
8. `c9d45beb2744835a79bc46ea7aa8d2e209f9139e`: keep the post-approval
57+
cancellation/deadline check inside the existing typed-error boundary.
58+
Self-review of patch 7 reproduced a raw LoginExpired escaping if approval
59+
finished after the deadline. The public regression in tests/test_login.py
60+
now gets AuthOperationError(reason="login_expired"), persists no credential,
61+
and can start a new login immediately. This patch is downstream only.
5662

5763
The first four patches remain necessary in v1.3.0. Its Meta login changes supply
5864
the minted endpoint and plan; they do not change bound selection, renewal

0 commit comments

Comments
 (0)