Skip to content

Fix inactive API configuration cleanup - #3

Open
mickeyslaven wants to merge 1 commit into
30hours:mainfrom
mickeyslaven:fix/inactive-request-cleanup
Open

mickeyslaven wants to merge 1 commit into
30hours:mainfrom
mickeyslaven:fix/inactive-request-cleanup

Conversation

@mickeyslaven

@mickeyslaven mickeyslaven commented Aug 27, 2026

Copy link
Copy Markdown

Fixes #2

Summary

  • use the stored request activity timestamp when removing inactive API configurations
  • preserve the existing cleanup timeout and request-limit behavior

Testing

  • reproduced the issue locally by filling the configuration table, waiting 35 seconds, and confirming a new configuration still returned HTTP 400
  • repeated the same test with this fix and confirmed the new configuration returned HTTP 200 after 35 seconds
  • node --check src/server.js
  • git diff --check

@mickeyslaven
mickeyslaven marked this pull request as ready for review August 27, 2026 10:53
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.

Inactive API configs are never removed

1 participant