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

Add Content-Location header #870

Closed
Stebalien opened this issue Jul 22, 2016 · 1 comment
Closed

Add Content-Location header #870

Stebalien opened this issue Jul 22, 2016 · 1 comment
Labels
A-headers Area: headers. E-easy Effort: easy. A task that would be a great starting point for a new contributor.

Comments

@Stebalien
Copy link
Contributor

See https://tools.ietf.org/html/rfc7231#section-3.1.4.2

@seanmonstar seanmonstar added A-headers Area: headers. E-easy Effort: easy. A task that would be a great starting point for a new contributor. labels Jul 23, 2016
@koivunej
Copy link
Contributor

I'd like to take a shot at this. ABNF from RFC here is:

Content-Location = absolute-URI / partial-URI

ABNF is exactly same as with Referer which is implemented as

header! {
    // TODO Use URL
    (Referer, "Referer") => [String]

    // testcase
}

/TODO:? Use URL/ is mentioned for Location and Referer headers, so I guess it'd be best to implement Content-Location as simple (ContentLocation, "Content-Location") => [String] for consistency?

koivunej added a commit to koivunej/hyper that referenced this issue Jul 27, 2016
koivunej added a commit to koivunej/hyper that referenced this issue Jul 27, 2016
seanmonstar pushed a commit that referenced this issue Jul 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-headers Area: headers. E-easy Effort: easy. A task that would be a great starting point for a new contributor.
Projects
None yet
Development

No branches or pull requests

3 participants