Skip to content

Bound SEA auth retry counts - #1422

Open
ming1523 wants to merge 2 commits into
amark:masterfrom
ming1523:codex/sea-auth-retries-guard
Open

Bound SEA auth retry counts#1422
ming1523 wants to merge 2 commits into
amark:masterfrom
ming1523:codex/sea-auth-retries-guard

Conversation

@ming1523

Copy link
Copy Markdown

Summary

  • normalize SEA auth retry counts to finite non-negative integers
  • preserve the default retry budget and retries=0 behavior
  • add regression coverage for default, zero, one, negative, fractional, and infinite values

Validation

  • node --check sea.js
  • node --check sea/auth.js
  • node --check test/sea/sea.js
  • direct auth control-flow smoke test: all six cases completed with an error callback
  • npm run testsea -- --grep "auth retry bounds" could not start because mocha is not installed in the environment

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.

1 participant