Skip to content

[WebAuthn] Return attachment=cross-platform for security key creates/assertions#50796

Merged
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
pascoej:eng/Return-attachment-cross-platform-for-security-key-creates-assertions
Sep 17, 2025
Merged

[WebAuthn] Return attachment=cross-platform for security key creates/assertions#50796
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
pascoej:eng/Return-attachment-cross-platform-for-security-key-creates-assertions

Conversation

@pascoej
Copy link
Member

@pascoej pascoej commented Sep 16, 2025

93c6967

[WebAuthn] Return attachment=cross-platform for security key creates/assertions
rdar://156414985
https://bugs.webkit.org/show_bug.cgi?id=298932

Reviewed by Charlie Wolfe.

Whenever we switched over to using AuthenticationServices API, we stopped setting
attachment=cross-platform for security key requests. This patch fixes that.

Unfortunately we cannot presently test the path that this change takes. We should be
able to once we extend AuthenticationServices testing framework to set a mock configuration
from within the AuthenticationServicesAgent process. That work is represented by https://bugs.webkit.org/show_bug.cgi?id=298932.

* Source/WebKit/UIProcess/WebAuthentication/Cocoa/WebAuthenticatorCoordinatorProxy.mm:
(WebKit::fromASAuthorizationPublicKeyCredentialAttachment):
(WebKit::WebAuthenticatorCoordinatorProxy::performRequest):

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

fc0d21f

Misc iOS, visionOS, tvOS & watchOS macOS Linux Windows Apple Internal
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe ✅ 🛠 win ⏳ 🛠 ios-apple
✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug ❌ 🧪 wpe-wk2 ✅ 🧪 win-tests ⏳ 🛠 mac-apple
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 ✅ 🧪 api-mac ✅ 🧪 api-wpe ⏳ 🛠 vision-apple
✅ 🧪 ios-wk2-wpt ✅ 🛠 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

@pascoej pascoej requested a review from cdumez as a code owner September 16, 2025 08:51
@pascoej pascoej self-assigned this Sep 16, 2025
@pascoej pascoej force-pushed the eng/Return-attachment-cross-platform-for-security-key-creates-assertions branch from d867e8e to fc0d21f Compare September 17, 2025 09:18
@pascoej pascoej added the merge-queue Applied to send a pull request to merge-queue label Sep 17, 2025
…assertions

rdar://156414985
https://bugs.webkit.org/show_bug.cgi?id=298932

Reviewed by Charlie Wolfe.

Whenever we switched over to using AuthenticationServices API, we stopped setting
attachment=cross-platform for security key requests. This patch fixes that.

Unfortunately we cannot presently test the path that this change takes. We should be
able to once we extend AuthenticationServices testing framework to set a mock configuration
from within the AuthenticationServicesAgent process. That work is represented by https://bugs.webkit.org/show_bug.cgi?id=298932.

* Source/WebKit/UIProcess/WebAuthentication/Cocoa/WebAuthenticatorCoordinatorProxy.mm:
(WebKit::fromASAuthorizationPublicKeyCredentialAttachment):
(WebKit::WebAuthenticatorCoordinatorProxy::performRequest):

Canonical link: https://commits.webkit.org/300084@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/Return-attachment-cross-platform-for-security-key-creates-assertions branch from fc0d21f to 93c6967 Compare September 17, 2025 15:43
@webkit-commit-queue
Copy link
Collaborator

Committed 300084@main (93c6967): https://commits.webkit.org/300084@main

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

@webkit-commit-queue webkit-commit-queue merged commit 93c6967 into WebKit:main Sep 17, 2025
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Sep 17, 2025
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.

4 participants