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

Handle exception if body is already parsed #33

Closed
nathanclevenger opened this issue Oct 10, 2022 · 2 comments
Closed

Handle exception if body is already parsed #33

nathanclevenger opened this issue Oct 10, 2022 · 2 comments

Comments

@nathanclevenger
Copy link
Member

If CTX.do is called twice, and there is a body, we need to more elegantly handle the situation. Maybe we want to default to cloning the request so the body stream isn't read and destroyed.

@nathanclevenger
Copy link
Member Author

This is definitely an issue if we're passing the request into any other service binding or durable object, so we have to resolve this one first.

@sam-lippert
Copy link
Contributor

#38

@sam-lippert sam-lippert closed this as not planned Won't fix, can't repro, duplicate, stale Oct 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants