-
Notifications
You must be signed in to change notification settings - Fork 46.9k
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
Permit th headers
attribute
#2553
Conversation
👍 Can you remove the commit with the documentation update? We'll take care of that when we do the release (this makes it easier to cherry pick into stable branches) |
Oh also, did you test that |
@zpao I confirm that |
abc23fc
to
9a472ca
Compare
@zpao I just removed the doc commit, so it's simply the |
@mhuggins yea, we just manual test it for now because behavior hasn't been consistent. We may make this better in the future. Thanks btw! |
Permits the headers attribute as defined by the W3C. Resolves #2516.