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.

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

20240424 webauthn idls #3932

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Firstyear
Copy link

Update Webauthn IDLs to Level 3

@daxpedda
Copy link
Collaborator

Checking the spec I can see that its still in the draft stage. We usually make those APIs unstable.

We made exceptions in the past for APIs that are unlikely to change e.g. because they are so widely used that they can't change without breaking the web, which I assume is not the case here.

@Firstyear
Copy link
Author

They are pretty unlikely to change though, since a lot of is it based on webauthn-l2. Regardless, we can just wait I guess?

@daxpedda
Copy link
Collaborator

They are pretty unlikely to change though, since a lot of is it based on webauthn-l2.

Just looking at the repo it seems to me that there are still a lot of small details up for change.

Regardless, we can just wait I guess?

Moving it behind unstable is what we usually do.

@Firstyear
Copy link
Author

What's the right way to mark this "unstable" then?

@daxpedda
Copy link
Collaborator

Just move the WebIDL files from the enabled folder into the unstable one and run the generator again.

Let me know if you run into any problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants