Skip to content

need to reserve field name and method name "*" #274

Closed

Description

@reschke

A field name of "*" would clash with the definition of "Vary" (see #272).

A method name of "*" would clash with Access-Control-Request-Method (see https://fetch.spec.whatwg.org/#http-new-header-syntax) (ack @annevk)

Activity

  1. changed the title [-]need to reserved field name and method name "*"[/-] [+]need to reserve field name and method name "*"[/+] on Jan 9, 2020
  2. reschke commented on Jan 10, 2020

    @reschke
    ContributorAuthor

    ...or disallow registration.

  3. reschke commented on Feb 2, 2020

    @reschke
    ContributorAuthor

    partly blocked by #273

  4. reschke commented on Jun 5, 2020

    @reschke
    ContributorAuthor

    To "reserve" in the method registry, we'll need to add a "comments" column to the registry. Makes sense?

    cc: @mnot

  5. annevk commented on Jun 5, 2020

    @annevk
    Contributor

    Would you leave Safe and Idempotent empty as well? Forbidding registration might be simpler?

  6. reschke commented on Jun 5, 2020

    @reschke
    ContributorAuthor

    Good question. Maybe "n/a"?

    In any case, extending the registration procedure to disallow "*" works for me as well.

  7. mnot commented on Jun 9, 2020

    @mnot
    Member

    Disallowing registration is fine for me too. Should we also disallow registering it as a field name?

    There are a few other ways we could do this that might be worth considering; e.g., we could require registered values to have at least two characters, or require registered values to begin with ALPHA. We already say:

    While the field-name syntax is defined to allow any token character, in practice some implementations place limits on the characters they accept in field-names. To be interoperable, new field names should constrain themselves to alphanumeric characters, "-", and ".", and should begin with an alphanumeric character.

  8. reschke commented on Jun 9, 2020

    @reschke
    ContributorAuthor

    Disallowing registration is fine for me too. Should we also disallow registering it as a field name?

    Yes, for consistency.

  9. mnot commented on Jul 2, 2020

    @mnot
    Member

    Discussed; put into the registry with a comment about why it's reserved; not safe or idempotent.

  10. added a commit that references this issue on Jul 10, 2020
  11. reschke commented on Jul 10, 2020

    @reschke
    ContributorAuthor
  12. added 2 commits that reference this issue on Jul 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions