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

Example 4 in 1.3.3 uses txAuthSimple #1401

Closed
ve7jtb opened this issue Apr 1, 2020 · 2 comments · Fixed by #1426
Closed

Example 4 in 1.3.3 uses txAuthSimple #1401

ve7jtb opened this issue Apr 1, 2020 · 2 comments · Fixed by #1426
Assignees
@ve7jtb
Labels
type:editorial
Milestone
L2-WD-03 Feature ...

Comments

@ve7jtb
Copy link
Contributor

ve7jtb commented Apr 1, 2020

We could leave the example as txAuthSimple is a registered extension, however that may not be as helpful to developers as one might hope.

We could rework the example to use another extension, or remove the extension entirely.

It just seems thrown in there and not really relevant to the example.
@nadalin nadalin added this to the L2-WD-03 milestone Apr 1, 2020
@selfissued
Copy link
Contributor

I think we should rework the example to use an extension in the spec - perhaps AppID.

@equalsJeffH
Copy link
Contributor

on 2020-04-01 call: we need to re-work the example to use one of the remaining extensions. @sbweeden suggests using credProps. others suggest appID. @ve7jtb will do something and we can complain about it....

@ve7jtb ve7jtb linked a pull request May 27, 2020 that will close this issue
Update examples to use current Extensions #1426
Merged
@emlun emlun mentioned this issue May 27, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ve7jtb ve7jtb

Labels
type:editorial
Projects
None yet
Milestone
L2-WD-03 Feature Complete
Development

Successfully merging a pull request may close this issue.

Update examples to use current Extensions w3c/webauthn
4 participants
@ve7jtb @equalsJeffH @selfissued @nadalin