Bug 64789 - `<a ping>`'s `Ping-From` implementation doesn't match the spec.
Summary: `<a ping>`'s `Ping-From` implementation doesn't match the spec.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Page Loading (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Minor
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-19 02:47 PDT by Mike West
Modified: 2011-07-19 14:56 PDT (History)
4 users (show)

See Also:


Attachments
Patch for review. (4.07 KB, patch)
2011-07-19 02:55 PDT, Mike West
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mike West 2011-07-19 02:47:43 PDT
According to http://www.whatwg.org/specs/web-apps/current-work/multipage/links.html#hyperlink-auditing `<a ping>` should send `Ping-From` HTTP header in addition to a `referer` header when pinging an origin distinct from the initial document (when the initial document wasn't loaded over HTTPS.

Currently, WebKit only sends the referrer header.

I'm not really convinced that the spec is correct here, as the additional header seems completely redundant, but we should probably follow the letter of the spec here. Patch for review in a moment.
Comment 1 Mike West 2011-07-19 02:55:17 PDT
Created attachment 101292 [details]
Patch for review.
Comment 2 Nate Chapin 2011-07-19 12:07:12 PDT
Comment on attachment 101292 [details]
Patch for review.

Did the spec change?  I was pretty sure that the spec said Ping-From and Referer were mutually exclusive when ping was implemented in WebKit.

I agree that this patch matches the behavior as it's currently specced, though.
Comment 3 Mike West 2011-07-19 14:08:15 PDT
Comment on attachment 101292 [details]
Patch for review.

Alright, if we agree that this matches the spec, then I'd ask you to throw this into the commit queue for me.

Thanks!
Comment 4 WebKit Review Bot 2011-07-19 14:56:51 PDT
Comment on attachment 101292 [details]
Patch for review.

Clearing flags on attachment: 101292

Committed r91306: <http://trac.webkit.org/changeset/91306>
Comment 5 WebKit Review Bot 2011-07-19 14:56:56 PDT
All reviewed patches have been landed.  Closing bug.