W3C home > Mailing lists > Public > public-webauthn@w3.org > September 2017

RE: [webauthn] new commits pushed by leshi

From: Mike Jones <Michael.Jones@microsoft.com>
Date: Wed, 13 Sep 2017 20:21:46 +0000
To: Alexei Czeskis <aczeskis@google.com>
CC: Alexei Czeskis via GitHub <sysbot+gh@w3.org>, "public-webauthn@w3.org" <public-webauthn@w3.org>
Message-ID: <CY4PR21MB0504EE3D4E1E835E498612A4F56E0@CY4PR21MB0504.namprd21.prod.outlook.com>
Were the PRs reviewed on today’s call? I still think that using different names for the same things in different places is a big step in the wrong direction.

From: Alexei Czeskis<mailto:aczeskis@google.com>
Sent: Wednesday, September 13, 2017 1:18 PM
To: Mike Jones<mailto:Michael.Jones@microsoft.com>
Cc: Alexei Czeskis via GitHub<mailto:sysbot+gh@w3.org>; public-webauthn@w3.org<mailto:public-webauthn@w3.org>
Subject: Re: [webauthn] new commits pushed by leshi

These were all merged into master as part of PRs, not directly.


Thanks!
-Alexei


________________

            . Alexei Czeskis .:. Securineer .

On Wed, Sep 13, 2017 at 1:14 PM, Mike Jones <Michael.Jones@microsoft.com<mailto:Michael.Jones@microsoft.com>> wrote:
s/our a lot of effort/put a lot of effort/

From: Mike Jones<mailto:Michael.Jones@microsoft.com>
Sent: Wednesday, September 13, 2017 1:07 PM
To: Alexei Czeskis via GitHub<mailto:sysbot+gh@w3.org>; public-webauthn@w3.org<mailto:public-webauthn@w3.org>
Subject: RE: [webauthn] new commits pushed by leshi


I'm looking at this from a phone and so may be confused, but it looks to me like these changes were pushed to master rather than put into a PR for review. If so, please revert these changes.



We have our a lot of effort into keeping the names in WebAuthn in sync with CTAP. This change is an unnecessary departure from this policy, which will confuse readers for no good reason.



From: Alexei Czeskis via GitHub<mailto:sysbot+gh@w3.org>
Sent: Wednesday, September 13, 2017 10:20 AM
To: public-webauthn@w3.org<mailto:public-webauthn@w3.org>
Subject: [webauthn] new commits pushed by leshi



The following commits were just pushed by leshi to https://github.com/w3c/webauthn:

* using descriptive names for authenticator selection criteria (#555)

Since we don't directly send the values of the AuthenticatorSelectionCriteria dictionary keys over the wire, it's ok to leave these values in a human-readable form.
  by balfanz
https://github.com/w3c/webauthn/commit/dcf793928221b1883f4c9ac4dd5264b570606e52
Received on Wednesday, 13 September 2017 20:22:12 UTC

This archive was generated by hypermail 2.4.0 : Tuesday, 5 July 2022 07:26:27 UTC