fix: websocket prefix for StreamingResponse and FileResponse denial (… #3085

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.

Sign up for GitHub

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

Jump to bottom

Open

rahuldevikar wants to merge 1 commit into Kludex:main from rahuldevikar:users/rahuldevikar/bug/fix3048

+92 −26

rahuldevikar commented Dec 29, 2025

Summary

Allow StreamingResponse to be used with websocket.send_denial_response()
Allow FileResponse to be used with websocket.send_denial_response()
Add proper ASGI message type prefixes when used in WebSocket context
Add comprehensive tests for both response types with denial responses
Fixes Can not use send_denial_response with StreamingResponse #3048"

Checklist

[c] I understand that this PR may be closed in case there was no previous discussion. (This doesn't apply to typos!)
[c] I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
[c] I've updated the documentation accordingly.


          fix: websocket prefix for StreamingResponse and FileResponse denial (K…

04e21a8

…ludex#3048)

rahuldevikar mentioned this pull request

Can not use send_denial_response with StreamingResponse #3048

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet