-
Notifications
You must be signed in to change notification settings - Fork 212
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
Should frameworks change scope values? #343
Comments
Maybe relevant: encode/starlette#1336. |
Yeah, you're not really meant to change the value of path, I feel - since it's meant to always have root_path at the start. I'm not going to go all out and say something absolutely shouldn't, but I think if it's modified then root_path has to be included in that to make things make sense. |
We'll fix Starlette. It's not Hypercorn's fault. Uvicorn has the same behavior as Hypercorn. 🙏 |
Related: Starlette at least mutates |
See pgjones/hypercorn#75 where Starlette changes the
scope["path"]
, should ASGI frameworks do this or consider the existing values immutable? (I don't think the specification says - I may work around by passing a copy to the framework)The text was updated successfully, but these errors were encountered: