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

Allow setting HTTP(S) headers in beacon #30

Closed
bhamiltoncx opened this issue May 2, 2016 · 3 comments
Closed

Allow setting HTTP(S) headers in beacon #30

bhamiltoncx opened this issue May 2, 2016 · 3 comments
Milestone

Comments

@bhamiltoncx
Copy link

Currently, the Beacon standard doesn't allow specifying HTTP headers in the request (except for Content-Type when the Blob API is used).

We'd like to specify additional headers in the HTTP beacon (for example, when using zero-rating to indicate to ISPs that a request should not count against a user's data transmission quota).

This would require updating:

  1. § 3.1 (sendBeacon Method) — add additional optional parameter httpHeaders to include object whose key-value pairs are included in the request
  2. § 3.2 (Processing Model) — update headerList logic to start out by populating list from httpHeaders, if non-null
@toddreifsteck
Copy link
Member

#27 is our current proposal for adding Headers/Method. I need to complete that PR and then browsers need to implement, but that is likely where this will be solved.

@bhamiltoncx
Copy link
Author

Great!
2016年5月11日(水) 22:16 Todd Reifsteck [email protected]:

#27 #27 is our current proposal for
adding Headers/Method. I need to complete that PR and then browsers need to
implement, but that is likely where this will be solved.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#30 (comment)

@igrigorik igrigorik added this to the Level 2 milestone Jun 29, 2016
@igrigorik
Copy link
Member

@bhamiltoncx Fetch (spec) integration has landed: whatwg/fetch#388 (comment). Once that's implemented, you'll be able to set custom headers/methods/etc, via fetch.

Closing this bug.

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

3 participants