Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cleanup requestlogging middleware #308

Open
TTA777 opened this issue Dec 4, 2024 · 0 comments
Open

Cleanup requestlogging middleware #308

TTA777 opened this issue Dec 4, 2024 · 0 comments
Labels
quickpick Good for newcomers

Comments

@TTA777
Copy link
Member

TTA777 commented Dec 4, 2024

In order to solve a recent issue for the mobilepay webhook endpoint, I hastily implemented a requestLogging middleware. If we want to keep it, we should limit the scope of what is logged, and inspect the code further, since it was mostly AI generated.

For now, it is disabled using feature flags, and won't process requests unless re-enabled and the application restarted

@TTA777 TTA777 added the quickpick Good for newcomers label Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
quickpick Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant