-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
tvOS Cloud-Synced Auth #8878
Comments
@ncooke3 any news here? Is this even supposed to be working? |
Hi @nidegen, apologies for the delay. No updates as of now. I'll look into it this week and report back. |
No worries, thank you! |
I filed an internal b/212482016 to track this. |
nice! @weixifan so you can confirm this is not only an issue on my side, right? |
Hi guys, how is it looking? Can you confirm this is a real bug? |
Hi @nidegen, sincere apologies for the delay. I'm working on reproducing the issue today. I have some questions:
|
No worries! Unfortunately I am pretty sure everything is correct. I have it running for iOS and macOS. The thing is however that I am not sure if it is at all possible on tvOS. Some forum posts indicate that it could be unsupported: https://developer.apple.com/forums/thread/690714 So, I am still not sure if I should expect an error on my side or if I should give up:) |
P.S.: There is no error, just |
Hi @nidegen, I was able to reproduce the issue without using Firebase. I created three apps: an iOS app, a macOS app, and a tvOS app. I followed Apple's Sharing Access to Keychain Items Among a Collection of Apps documentation. I was able to share and sync password data between iOS and macOS but couldn't get tvOS to work either. The underlying error code I'm seeing from the Apple's Security framework is I reproduced on a tvOS simulator. Are you using a physical tvOS device or the simulator? |
Great but not great 😅 I tested both on a physical device but it seemed to behave the same. I tested with 8.11. as I hoped the macOS extension keychain fix would help on tvOS, but it remained the same.
… On 20 Jan 2022, at 22:32, Nick Cooke ***@***.***> wrote:
Hi @nidegen,
I was able to reproduce the issue without using Firebase. I created three apps: an iOS app, a macOS app, and a tvOS app. I followed Apple's Sharing Access to Keychain Items Among a Collection of Apps documentation.
I was able to share and sync password data between iOS and macOS but couldn't get tvOS to work either. The underlying error code I'm seeing from the Apple's Security framework is -25300. This code is an errSecItemNotFound which I believe explains why Firebase is returning a nil user in your case.
I reproduced on a tvOS simulator. Are you using a physical tvOS device or the simulator?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.
|
Ah, I was hoping that testing on a physical tvOS device would work but I guess not :( Thanks for the thorough investigation. It appears then that this issue isn't caused by Firebase and seems to be on Apple's side– which is quite confusing since Apple's documentation indicates that this should be supported. 🤔 I think the next step would be to file feedback with Apple as this seems like unexpected behavior. I'm going to close this issue but I'm happy to continue the conversation. |
Yes, that confirms my suspicion. Should I file an Apple feedback? Can you maybe share your MWE? |
I'm packaging up my MWE and will share it when ready. I'll also file a feedback. |
Hi @nidegen, I added my repro with instructions here: https://github.com/ncooke3/repros/tree/main/firebase-8878/KeychainSharing Would you mind confirming the behavior on a tvOS device? |
Thanks a lot @nidegen– I appreciate the help! I filed a radar with Apple on my end. Hopefully we can get an answer for the community about keychain sharing on tvOS. I'll keep this thread updated. |
Great, did the same. Hope it will get fixed soon. As a side note, while shared keychain is awesome I would be able to circumvent it if Phone Auth was available on tvOS too (macOS would be nice too). Would be great to see this happen too:) |
Unfortunately, Apple updated the docs indicating tvOS is not supposed to sync over iCloud:( https://developer.apple.com/forums/thread/690714?answerId=702729022#702729022 |
Thanks for the update! That is indeed unfortunate. Hopefully support is added in the future. 🤞 |
[REQUIRED] Step 1: Describe your environment
Swift Package Manager
[REQUIRED] Step 2: Describe the problem
Steps to reproduce:
Following this, I do share Auth over iCloud Keychain sync between iOS and macOS apps successfully.
However, if I try this with tvOS, it does not work. Does tvOS maybe not support Keychain Sharing?
Officially, tvOS supports it but there are some other users with issues.
I wonder if there could be an issue with Firebase's keychain access implementation.
Relevant Code:
The text was updated successfully, but these errors were encountered: