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

Normalization for parameters #282

Closed
martinthomson opened this issue Jan 17, 2017 · 8 comments
Closed

Normalization for parameters #282

martinthomson opened this issue Jan 17, 2017 · 8 comments

Comments

@martinthomson
Copy link
Contributor

Are the following header fields considered equal or equivalent or are they different?

Header: value; p=pval
Header: value; p="pval"
Header: value; p="Pval"
Header: value; P="Pval"

Some specializations of the header syntax permit both quoted-string and token for parameter values. If the string value after removal of quotes is the same as the token, is it the same value?

What about case folding for parameter names? Do we do that?

@reschke
Copy link
Contributor

reschke commented Jan 17, 2017

  1. and 2) should be the same (token vs quoted string)

  2. is different

  3. might be the same as 3), there are at least some pre-existing header fields that treat parameter names case-insensitively.

@martinthomson
Copy link
Contributor Author

That was my guess too, but the more pertinent question is: where is that written down?

@reschke
Copy link
Contributor

reschke commented Jan 17, 2017

We need to write it down.

@bsdphk
Copy link
Contributor

bsdphk commented Feb 1, 2017

I can see a point of automatically "promoting" an ascii-string to identifier if removing the quotes would make it one.

For the remaining questions I think the individual header definition should sort that out.

@mnot
Copy link
Member

mnot commented Dec 13, 2017

I think this is OBE - @martinthomson can we close?

@martinthomson
Copy link
Contributor Author

I think that the question remains relevant, but are you suggesting that I make it a 723x-bis question instead?

@mnot
Copy link
Member

mnot commented Jan 12, 2018

Well, if you want it to be addressed for token and quoted-string in parameter, it needs to be raised against httpter, yes. If you want it to be addressed within structured headers, I think it is OBE.

@martinthomson
Copy link
Contributor Author

I'll reopen on bisbisbis then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

4 participants