Skip to content

fix: reset requestTime on token acquisition failure#316

Merged
pyrooka merged 2 commits intomainfrom
nb/fix-vulnerability
Feb 25, 2026
Merged

fix: reset requestTime on token acquisition failure#316
pyrooka merged 2 commits intomainfrom
nb/fix-vulnerability

Conversation

@pyrooka
Copy link
Member

@pyrooka pyrooka commented Feb 19, 2026

Reset requestTime and clear pendingRequests in the error handler to prevent a 60-second authentication freeze after failed token acquisition attempts. Previously, requestTime remained set after failures, causing subsequent requests to queue indefinitely.

Checklist
  • npm test passes (tip: npm run lint-fix can correct most style issues)
  • tests are included
  • documentation is changed or added

Reset requestTime and clear `pendingRequests` in the error handler
to prevent a 60-second authentication freeze after failed token
acquisition attempts. Previously, `requestTime` remained set after
failures, causing subsequent requests to queue indefinitely.

Signed-off-by: Norbert Biczo <pyrooka@users.noreply.github.com>
@pyrooka pyrooka self-assigned this Feb 19, 2026
@pyrooka pyrooka requested review from Andris28 and diatrcz February 19, 2026 12:55
Copy link
Contributor

@diatrcz diatrcz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@pyrooka pyrooka merged commit 6ea6f2b into main Feb 25, 2026
14 checks passed
@pyrooka pyrooka deleted the nb/fix-vulnerability branch February 25, 2026 11:17
ibm-devx-sdk pushed a commit that referenced this pull request Feb 25, 2026
## [5.4.8](v5.4.7...v5.4.8) (2026-02-25)

### Bug Fixes

* **token-manager:** reset `requestTime` on token acquisition failure ([#316](#316)) ([6ea6f2b](6ea6f2b))
@ibm-devx-sdk
Copy link

🎉 This PR is included in version 5.4.8 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants