WebKit icon indicating copy to clipboard operation
WebKit copied to clipboard

Use different keychain attribute for passkey credential ID

Open aznair opened this issue 1 year ago โ€ข 13 comments

4c612ce1f2bb816614e7af2d1cdc0b6847d403f1

Use different keychain attribute for passkey credential ID
https://bugs.webkit.org/show_bug.cgi?id=281344
rdar://137771569

Reviewed by NOBODY (OOPS!).

Use kSecAttrAlias going forward, and if a query using this field fails then fall
back to kSecAttrApplicationLabel

* Source/WebKit/UIProcess/API/Cocoa/_WKWebAuthenticationPanel.mm:
(getAllLocalAuthenticatorCredentialsImpl):
(+[_WKWebAuthenticationPanel deleteLocalAuthenticatorCredentialWithGroupAndID:credential:]):
(+[_WKWebAuthenticationPanel setDisplayNameForLocalCredentialWithGroupAndID:credential:displayName:]):
(+[_WKWebAuthenticationPanel setNameForLocalCredentialWithGroupAndID:credential:name:]):
(+[_WKWebAuthenticationPanel importLocalAuthenticatorWithAccessGroup:credential:error:]):
(+[_WKWebAuthenticationPanel exportLocalAuthenticatorCredentialWithGroupAndID:credential:error:]):
* Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalAuthenticator.mm:
(WebKit::LocalAuthenticatorInternal::produceHashSet):
(WebKit::LocalAuthenticator::getExistingCredentials):
(WebKit::LocalAuthenticator::processLargeBlobExtension):
(WebKit::LocalAuthenticator::continueMakeCredentialAfterUserVerification):
(WebKit::LocalAuthenticator::continueGetAssertionAfterUserVerification):
(WebKit::LocalAuthenticator::receiveException const):
(WebKit::LocalAuthenticator::deleteDuplicateCredential const):
* Tools/WebKitTestRunner/cocoa/TestControllerCocoa.mm:
(WTR::TestController::cleanUpKeychain):
(WTR::TestController::keyExistsInKeychain):

https://github.com/WebKit/WebKit/commit/e006883c9bf369619f91e1804db403d8184054a3

Misc iOS, visionOS, tvOS & watchOS macOS Linux Windows
โœ… ๐Ÿงช style โœ… ๐Ÿ›  ios โœ… ๐Ÿ›  mac โณ ๐Ÿ›  wpe โณ ๐Ÿ›  win
โœ… ๐Ÿงช bindings โœ… ๐Ÿ›  ios-sim โœ… ๐Ÿ›  mac-AS-debug โณ ๐Ÿงช wpe-wk2 โณ ๐Ÿงช win-tests
โœ… ๐Ÿงช webkitperl โณ ๐Ÿงช ios-wk2 loading ๐Ÿงช api-mac โณ ๐Ÿงช api-wpe
loading ๐Ÿงช webkitpy โณ ๐Ÿงช ios-wk2-wpt โœ… ๐Ÿงช mac-wk1 โณ ๐Ÿ›  wpe-cairo
loading ๐Ÿ›  ๐Ÿงช jsc โณ ๐Ÿงช api-ios loading ๐Ÿงช mac-wk2 โœ… ๐Ÿ›  gtk
loading ๐Ÿ›  ๐Ÿงช jsc-arm64 loading ๐Ÿ›  vision โณ ๐Ÿงช mac-AS-debug-wk2 โณ ๐Ÿงช gtk-wk2
loading-orange ๐Ÿ›  vision-sim โœ… ๐Ÿงช mac-wk2-stress โณ ๐Ÿงช api-gtk
โณ ๐Ÿงช vision-wk2 โณ ๐Ÿงช mac-intel-wk2 โœ… ๐Ÿ›  jsc-armv7
โœ… ๐Ÿ›  tv โณ ๐Ÿ›  mac-safer-cpp โŒ ๐Ÿงช jsc-armv7-tests
โณ ๐Ÿ›  tv-sim
โณ ๐Ÿ›  watch
โณ ๐Ÿ›  watch-sim

aznair avatar Oct 11 '24 21:10 aznair

Converting to draft while I make some adjustments that have become necessary due to changes in Keychain.

aznair avatar Oct 15 '24 17:10 aznair

Committed 285575@main (1dc9ae6c1864): https://commits.webkit.org/285575@main

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

webkit-commit-queue avatar Oct 23 '24 00:10 webkit-commit-queue