🚀 Auto-PR: Merge test → main#794
Merged
Merged
Conversation
Test Suite: Passed: 62 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/26909260071
The pre-push hook already enforces version consistency via version:sync before every push. This CI check was always failing because the dynamic build suffixes (timestamp + SHA) cause expected mismatches between branches during PR validation.
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/26909525965
🧪 Auto-PR: Merge `develop` → `test`
Test Suite: Passed: 62 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/26909709104
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/26909340977
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/26909380551
The auth and email event loggers each requested a new client_credentials token from Keycloak every 60 seconds, generating 2 CLIENT_LOGIN events per minute (120/hour) that flood the OAuth monitoring dashboard. Add a static token cache shared across all BaseEventsLogger instances. The token is reused until 30s before expiry, reducing Keycloak token requests from ~120/hour to ~12/hour (once per 5-min token lifetime).
🧪 Auto-PR: Merge `develop` → `test`
Test Suite: Passed: 62 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/26917933708
Test Suite: Passed: 62 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/26918105524
Test Suite: Passed: 62 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/26917954482
The adminAuditPlugin's onAfterResponse and onBeforeHandle hooks were
using Elysia's default 'local' scope, meaning they only applied to
routes defined WITHIN the plugin itself (which has none). Since the
actual admin routes are defined in the parent instance (adminRoutes),
the hooks never fired — resulting in an empty Admin Audit Dashboard.
Fix: add { as: 'scoped' } to both hooks so they propagate up to the
parent instance where the admin routes are registered.
Only mirror warn/error to console by default. Info and debug logs are still buffered in localStorage and only printed to console when debug mode is active (?debug=1 or window.__DEBUG_ENABLED__ = true).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated Pull Request 🤖
This PR was automatically created to merge changes from
testintomain.Changes:
Review: Please review the changes before merging.
Last updated: 2026-06-03 23:14:12 UTC