Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add two new transport types to section 5.10.4 #1261

Closed
ve7jtb opened this issue Jul 17, 2019 · 2 comments · Fixed by #1264
Closed

Add two new transport types to section 5.10.4 #1261

ve7jtb opened this issue Jul 17, 2019 · 2 comments · Fixed by #1264
Assignees
@ve7jtb
Labels
stat:pr-open subtype:algorithms/WebIDL type:technical
Milestone
L2-WD-02

Comments

@ve7jtb
Copy link
Contributor

ve7jtb commented Jul 17, 2019

In order for RP to know if a credential can be used on a iOS device, we need to add new transport enumerators to Section 5.10.4.

We need one for the physical lightning connector, and one for the HID transport encapsulation.

Currently, we have usb, ble, nfc, and internal.

Strings to add
lightning
tunnel (it could be more specific but I don't want to step on any trademarks for the External Accessory Framework.)

Are we missing an entry for caBLE?

This should be relatively uncontroversial, however agreeing on the sting value could take us months:)
@agl agl added this to the L2-WD-02 milestone Jul 17, 2019
@agl
Copy link
Contributor

agl commented Jul 17, 2019

From call of 2019-07-17: throw name suggestions here. I suggest “tunnel” → tunnel-usb or tunnel-hid

@ve7jtb
Copy link
Contributor Author

ve7jtb commented Jul 24, 2019

In further conversations with AGL on the use of transports, we settled on a single value "lightning"

lightning
Indicates the respective authenticator can be contacted over removable Lightning.

This was referenced Jul 24, 2019
Closed
Merged
akshayku pushed a commit that referenced this issue Aug 28, 2019
@ve7jtb @akshayku
Add lightning transport (#1264)
30d3e9f 
* Update index.bs

Add lightning transport

Fixes #1261

* Update index.bs

Fix missing comma
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ve7jtb ve7jtb

Labels
stat:pr-open subtype:algorithms/WebIDL type:technical
Projects
None yet
Milestone
L2-WD-02
Development

Successfully merging a pull request may close this issue.

Add lightning transport
Add lightning to Transports
4 participants
@agl @ve7jtb @equalsJeffH @nadalin