Re: Extensions PR

in case it's useful, here's a direct link to the spec source diff..

  https://github.com/w3c/webauthn/pull/130/files


On 6/15/16, 8:50 AM, "Vijay Bharadwaj" <vijaybh@microsoft.com<mailto:vijaybh@microsoft.com>> wrote:

Hi folks,

Apologies for the delay on this - I had a few urgent things intervene in the past week.

I just send out the PR below for extensions, and would like to walk through it on today's call. I was surprised at how little change was involved, and how much of it was simply resolving conflicts between text that said one thing and other text in the same section that said the opposite. In general, I tried to remove uncertainty where possible (e.g. by being explicit that clients passing through extensions is a valid choice).

Here is a PDF of the updated document for ease of review. All changes are in Sections 6 and 7, but I think it may be better to simply read through those sections for consistency rather than only look for changes since the latter may overlook internal contradictions in the text.

-----Original Message-----
From: noreply@github.com<mailto:noreply@github.com> [mailto:noreply@github.com]
Sent: Wednesday, June 15, 2016 8:40 AM
To: public-webauthn@w3.org<mailto:public-webauthn@w3.org>
Subject: [w3c/webauthn] c05d78: Clean up extensions section

  Branch: refs/heads/vgb-extension-opt
  Home:   https://github.com/w3c/webauthn
  Commit: c05d78629156b9b99faedcd03151cc8cefa7ce91
      https://github.com/w3c/webauthn/commit/c05d78629156b9b99faedcd03151cc8cefa7ce91
  Author: Vijay Bharadwaj <vijaybh@users.noreply.github.com<mailto:vijaybh@users.noreply.github.com>>
  Date:   2016-06-15 (Wed, 15 Jun 2016)

  Changed paths:
    M index.src.html

  Log Message:
  -----------
  Clean up extensions section

- Resolve inconsistent wording in different parts of the section: fix places that incorrectly suggested extensions cannot be used with makeCredential, and resolve conflict between wording that said client arguments are required vs. not.
- Treat implementations that choose to pass through extensions as a first-class citizen by requiring that extensions be defined to allow for this. Fixes #97.
- Fix all the predefined extensions to conform to the updated guidelines.
- Minor issues: AAGUID should be binary not string (see #61, fixes #51) and encode true as Boolean in CBOR instead of forcing it to integer value 1.