bedrock-claude-chat icon indicating copy to clipboard operation
bedrock-claude-chat copied to clipboard

[BUG] Token validation in middleware

Open cyruxOP opened this issue 1 year ago • 2 comments

Describe the bug The middleware function while calling get_current_user is not catching any errors returned by get_current_user function.

To Reproduce Go to bedrock-claude-chat/backend/app/main.py Line 94 middleware function . And in the same parent folder dependencies.py when it raises error it is not being handled in the main.py middleware function. Which would lead to errors and multiple invocations.

Ideally when something fails we should return from middleware only without processing further.

cyruxOP avatar Sep 04 '24 04:09 cyruxOP

Please do not ignore the template and provide details. thank you

statefb avatar Sep 05 '24 00:09 statefb

@statefb Added more details

cyruxOP avatar Sep 05 '24 04:09 cyruxOP

This issue has been labeled as "stale" due to no response by the reporter within 1 month (and 14 days after last commented by someone). And it will be closed automatically 14 days later if not responded.

github-actions[bot] avatar Oct 06 '24 01:10 github-actions[bot]

This issue has been closed due to no response within 14 days after labeled as "stale", 14 days after last reopened, and 14 days after last commented.

github-actions[bot] avatar Oct 21 '24 01:10 github-actions[bot]