-
Notifications
You must be signed in to change notification settings - Fork 338
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
fix: detect preflight requests #1031
Conversation
4cb6d6d
to
0ef82a7
Compare
0ef82a7
to
d572eef
Compare
aa963b5
to
1dfa8fa
Compare
Thank you this helps us a lot in our development of data driven UIs 😄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we have a counter-example harness test, which tests an actual preflight request?
With that condition, it looks good to me.
I added 2 comments though, I know they are inherited code, still something to consider.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just curious about the Content-length
case mentioned by @marcelltoth
fe96602
to
8c8fb11
Compare
8c8fb11
to
89cfb07
Compare
Closes #1020