Skip to content

fix grammar for "Vary" header field #272

Closed

Description

@reschke

it uses

"*" / #vary

but "vary" is a token, and token allows "*" anyway. Also, see #7.

Activity

  1. self-assigned this
    on Jan 9, 2020
  2. reschke commented on Jan 9, 2020

    @reschke
    ContributorAuthor

    In particular, "*" is a valid field name. We probably should reserve it in the registry.

  3. annevk commented on Jan 9, 2020

    @annevk
    Contributor

    (CORS has an issue like that as well, including for a * method; see the note at https://fetch.spec.whatwg.org/#http-new-header-syntax.)

  4. reschke commented on Jan 10, 2020

    @reschke
    ContributorAuthor

    Proposal:

    vary-by = "*" / field-name
    Vary = 1#vary-by
    

    Then state that if "*" appears, no "real" field-name is allowed to appear as well (for backwards compat).

    Do we need to discuss cases like

    Vary: *, *
    

    ?

  5. royfielding commented on Feb 3, 2020

    @royfielding
    Member

    I don't see why that is an improvement.

  6. mnot commented on Feb 3, 2020

    @mnot
    Member

    I thought we were going to reserve * and define its meaning in prose...

  7. mnot commented on Feb 4, 2020

    @mnot
    Member

    I.e., it should just be

    Vary = 1#field-name

  8. reschke commented on Feb 4, 2020

    @reschke
    ContributorAuthor

    which implies registering "*" as reserved field name, right?

  9. mnot commented on Feb 4, 2020

    @mnot
    Member

    Yes. That's #274

  10. assigned and unassigned on Jul 2, 2020
  11. mnot commented on Jul 2, 2020

    @mnot
    Member

    Discussed; will create a PR to move ABNF and prose to list-based.

  12. added a commit that references this issue on Jul 8, 2020
  13. reschke commented on Jul 13, 2020

    @reschke
    ContributorAuthor

    Do we need to list that as change from 7230?

  14. mnot commented on Jul 14, 2020

    @mnot
    Member

    Probably a good idea; will commit to master and link here.

  15. added a commit that references this issue on Jul 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions