Skip to content

fix(sessions): validate long-poll window - #115

Merged
ralyodio merged 1 commit into
moshcoder:mainfrom
rissrice2105-agent:fix/session-poll-window
Jul 31, 2026
Merged

fix(sessions): validate long-poll window#115
ralyodio merged 1 commit into
moshcoder:mainfrom
rissrice2105-agent:fix/session-poll-window

Conversation

@rissrice2105-agent

Copy link
Copy Markdown
Contributor

Closes #114.

Summary

  • accept only positive decimal integers for SESSION_POLL_MS
  • reject partial, fractional, signed, scientific-notation, and out-of-range timer values
  • fall back to the safe 25-second window instead of letting Node clamp invalid delays to ~1 ms
  • preserve the short valid override used by integration tests

Validation

  • focused session tests: 8/8
  • full PWA suite: 147/147
  • git diff --check

@ralyodio
ralyodio merged commit 4f279be into moshcoder:main Jul 31, 2026
3 checks passed
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.

Malformed session poll windows create a hot request loop

2 participants