Skip to content

Digital Credentials: implement API using the Identity Document Services framework #46626

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

Conversation

marcoscaceres
Copy link
Contributor

@marcoscaceres marcoscaceres commented Jun 11, 2025

c4e6770

Digital Credentials: implement API using the Identity Document Services framework
https://bugs.webkit.org/show_bug.cgi?id=294237
rdar://151879131

Reviewed by Abrar Rahman Protyasha.

Implements the Digital Credentials API using the new Identity Document Services framework.

On iOS, this allows the user to select digital credentials from application that serve as document providers.
On macOS, credentiala selection cban be performed over CTAP with a compatible device by scanning a QR code.

Canonical link: https://commits.webkit.org/296741@main

a96a7d6

Misc iOS, visionOS, tvOS & watchOS macOS Linux Windows
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe ✅ 🛠 win
✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug ✅ 🧪 wpe-wk2 ⏳ 🧪 win-tests
✅ 🧪 webkitperl ❌ 🧪 ios-wk2 ✅ 🧪 api-mac ✅ 🧪 api-wpe
✅ 🧪 ios-wk2-wpt ✅ 🧪 mac-wk1 ✅ 🛠 wpe-cairo
✅ 🧪 api-ios ✅ 🧪 mac-wk2 ✅ 🛠 gtk
✅ 🛠 vision ⏳ 🧪 mac-AS-debug-wk2 ✅ 🧪 gtk-wk2
✅ 🛠 vision-sim ✅ 🧪 mac-wk2-stress ✅ 🧪 api-gtk
✅ 🛠 🧪 merge ✅ 🧪 vision-wk2 ✅ 🧪 mac-intel-wk2 ✅ 🛠 playstation
✅ 🛠 tv ✅ 🛠 mac-safer-cpp
✅ 🛠 tv-sim
✅ 🛠 watch
✅ 🛠 watch-sim

@marcoscaceres marcoscaceres self-assigned this Jun 11, 2025
@marcoscaceres marcoscaceres added the New Bugs Unclassified bugs are placed in this component until the correct component can be determined. label Jun 11, 2025
@marcoscaceres marcoscaceres force-pushed the eng/Digital-Credentials-implement-API-using-the-Identity-Document-Services-framework branch from 226140d to d79d437 Compare June 17, 2025 02:37
@marcoscaceres marcoscaceres force-pushed the eng/Digital-Credentials-implement-API-using-the-Identity-Document-Services-framework branch from d79d437 to 825e56e Compare June 17, 2025 02:58
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Jun 17, 2025
@marcoscaceres marcoscaceres removed the merging-blocked Applied to prevent a change from being merged label Jun 19, 2025
@marcoscaceres marcoscaceres force-pushed the eng/Digital-Credentials-implement-API-using-the-Identity-Document-Services-framework branch from 825e56e to 21ab7ab Compare June 19, 2025 00:45
@marcoscaceres marcoscaceres force-pushed the eng/Digital-Credentials-implement-API-using-the-Identity-Document-Services-framework branch from 21ab7ab to e14311f Compare June 19, 2025 01:32
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Jun 19, 2025
@marcoscaceres marcoscaceres removed the merging-blocked Applied to prevent a change from being merged label Jun 19, 2025
@marcoscaceres marcoscaceres force-pushed the eng/Digital-Credentials-implement-API-using-the-Identity-Document-Services-framework branch from e14311f to 2e66740 Compare June 19, 2025 08:16
@marcoscaceres marcoscaceres force-pushed the eng/Digital-Credentials-implement-API-using-the-Identity-Document-Services-framework branch from 2e66740 to 8c4d2aa Compare June 19, 2025 09:20
@marcoscaceres marcoscaceres force-pushed the eng/Digital-Credentials-implement-API-using-the-Identity-Document-Services-framework branch from 8c4d2aa to 4ee79ac Compare June 19, 2025 09:22
@marcoscaceres marcoscaceres force-pushed the eng/Digital-Credentials-implement-API-using-the-Identity-Document-Services-framework branch from 4ee79ac to bcbc611 Compare June 19, 2025 09:25
@marcoscaceres marcoscaceres force-pushed the eng/Digital-Credentials-implement-API-using-the-Identity-Document-Services-framework branch from bcbc611 to cf0a49c Compare June 19, 2025 09:27
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Jun 19, 2025
@marcoscaceres marcoscaceres removed the merging-blocked Applied to prevent a change from being merged label Jun 25, 2025
@marcoscaceres marcoscaceres force-pushed the eng/Digital-Credentials-implement-API-using-the-Identity-Document-Services-framework branch from cf0a49c to 153d380 Compare June 25, 2025 06:34
@marcoscaceres marcoscaceres force-pushed the eng/Digital-Credentials-implement-API-using-the-Identity-Document-Services-framework branch from 153d380 to 6bdfefb Compare June 26, 2025 01:24
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Jun 26, 2025
@marcoscaceres marcoscaceres removed the merging-blocked Applied to prevent a change from being merged label Jun 26, 2025
@marcoscaceres marcoscaceres force-pushed the eng/Digital-Credentials-implement-API-using-the-Identity-Document-Services-framework branch from 6bdfefb to 095d714 Compare June 26, 2025 08:54
@aprotyas aprotyas self-requested a review June 27, 2025 01:22
Copy link
Member

@aprotyas aprotyas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with a couple of small comments, hopefully we can get @rr-codes's blessings with the WebKitSwift changes!

@marcoscaceres marcoscaceres changed the title Digital Credentials: implement API using the Identity Document Services framework tDigital Credentials: implement API using the Identity Document Services framework Jun 27, 2025
@marcoscaceres marcoscaceres force-pushed the eng/Digital-Credentials-implement-API-using-the-Identity-Document-Services-framework branch from 095d714 to 180623a Compare June 27, 2025 01:44
@aprotyas
Copy link
Member

@marcoscaceres minor typo in the commit title!

@marcoscaceres marcoscaceres changed the title tDigital Credentials: implement API using the Identity Document Services framework Digital Credentials: implement API using the Identity Document Services framework Jun 27, 2025
@marcoscaceres marcoscaceres force-pushed the eng/Digital-Credentials-implement-API-using-the-Identity-Document-Services-framework branch from 180623a to 8f67948 Compare June 27, 2025 02:44
@marcoscaceres marcoscaceres force-pushed the eng/Digital-Credentials-implement-API-using-the-Identity-Document-Services-framework branch from 8f67948 to a96a7d6 Compare June 27, 2025 02:46
@marcoscaceres marcoscaceres added the safe-merge-queue Applied to automatically send a pull-request to merge-queue after passing EWS checks label Jun 27, 2025
@marcoscaceres
Copy link
Contributor Author

@rr-codes's blessings with the WebKitSwift changes!

Going to safe merge queue this for now as it's been sitting for a while, but happy to make any additional changes as a followup.

@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Jun 27, 2025
@marcoscaceres marcoscaceres added merge-queue Applied to send a pull request to merge-queue and removed merging-blocked Applied to prevent a change from being merged safe-merge-queue Applied to automatically send a pull-request to merge-queue after passing EWS checks labels Jun 27, 2025
…es framework

https://bugs.webkit.org/show_bug.cgi?id=294237
rdar://151879131

Reviewed by Abrar Rahman Protyasha.

Implements the Digital Credentials API using the new Identity Document Services framework.

On iOS, this allows the user to select digital credentials from application that serve as document providers.
On macOS, credentiala selection cban be performed over CTAP with a compatible device by scanning a QR code.

Canonical link: https://commits.webkit.org/296741@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/Digital-Credentials-implement-API-using-the-Identity-Document-Services-framework branch from a96a7d6 to c4e6770 Compare June 27, 2025 19:09
@webkit-commit-queue
Copy link
Collaborator

Committed 296741@main (c4e6770): https://commits.webkit.org/296741@main

Reviewed commits have been landed. Closing PR #46626 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit c4e6770 into WebKit:main Jun 27, 2025
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Jun 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Bugs Unclassified bugs are placed in this component until the correct component can be determined.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants