Opened 10 years ago

Closed 8 years ago

Last modified 7 years ago

#195 closed design (fixed)

auth-param syntax

Reported by: mnot@… Owned by: julian.reschke@…
Priority: normal Milestone: 16
Component: p7-auth Severity: Active WG Document
Keywords: Cc:

Description (last modified by julian.reschke@…)

Credentials are defined in 2617 as

  credentials = auth-scheme #auth-param
  auth-param     = token "=" ( token | quoted-string )

however, Basic seems to be an exception:

  challenge   = "Basic" realm
  credentials = "Basic" basic-credentials
  basic-credentials = base64-user-pass

Note that it previously says:

For Basic, the framework above is utilized as follows

(weakly) implying that it's conformant to the framework.

Options:

  1. Note this and say that Basic is grandfathered in; i.e., future schemes MUST NOT have bare arguments
  2. Allow bare arguments in credentials (which would imply that ordering is significant)
  3. Admit that new schemes can pretty much define their own syntax.

Attachments (1)

195.diff (6.4 KB) - added by julian.reschke@… 8 years ago.
proposed patch

Download all attachments as: .zip

Change History (18)

comment:1 Changed 9 years ago by julian.reschke@…

comment:2 Changed 9 years ago by julian.reschke@…

  • Owner set to julian.reschke@…

comment:3 Changed 9 years ago by julian.reschke@…

From [998]:

Incorporate auth framework from RFC 2617; ack RFC 2617's authors, fix known auth-param erratum (see #195)(see #237)

comment:4 Changed 9 years ago by julian.reschke@…

  • Resolution set to incorporated
  • Status changed from new to closed

comment:5 Changed 8 years ago by julian.reschke@…

  • Milestone changed from unassigned to 12

comment:6 Changed 8 years ago by mnot@…

  • Resolution incorporated deleted
  • Status changed from closed to reopened

comment:7 Changed 8 years ago by mnot@…

  • Resolution set to fixed
  • Status changed from reopened to closed

comment:8 Changed 8 years ago by julian.reschke@…

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:10 Changed 8 years ago by julian.reschke@…

  • Milestone changed from 12 to 15

comment:11 Changed 8 years ago by mnot@…

  • Milestone changed from 15 to unassigned

unsetting milestone until we have more discussion.

comment:12 Changed 8 years ago by julian.reschke@…

  • Description modified (diff)

Changed 8 years ago by julian.reschke@…

proposed patch

comment:13 Changed 8 years ago by julian.reschke@…

see [1394]

comment:14 Changed 8 years ago by julian.reschke@…

  • Milestone changed from unassigned to 16
  • Resolution set to incorporated
  • Status changed from reopened to closed

comment:15 Changed 8 years ago by mnot@…

  • Resolution incorporated deleted
  • Status changed from closed to reopened

comment:16 Changed 8 years ago by mnot@…

  • Resolution set to fixed
  • Status changed from reopened to closed

comment:17 Changed 7 years ago by mnot@…

  • Severity changed from Candidate WG Document to Active WG Document
Note: See TracTickets for help on using tickets.