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

AppCheck duplicate binaries in firebase.app and firebase.app-check #726

Closed
robinryf opened this issue May 13, 2023 · 6 comments · Fixed by #729
Closed

AppCheck duplicate binaries in firebase.app and firebase.app-check #726

robinryf opened this issue May 13, 2023 · 6 comments · Fixed by #729

Comments

@robinryf
Copy link

Using the versions

"com.google.firebase.app": "11.0.0", "com.google.firebase.app-check": "11.0.0"

I get exceptions

GUID [e67484c6c4644e6ab030360a71bfa7da] for asset 'Packages/com.google.firebase.app-check/Firebase/Plugins/x86_64/FirebaseCppAppCheck.bundle.meta' conflicts with:
  'Packages/com.google.firebase.app/Firebase/Plugins/x86_64/FirebaseCppAppCheck.bundle.meta'
We can't assign a new GUID because the asset is in an immutable folder. The asset will be ignored.
GUID [b45f6eb22b224bd4a0a45d3ddc8b9c7d] for asset 'Packages/com.google.firebase.app-check/Firebase/Plugins/x86_64/FirebaseCppAppCheck.dll.meta' conflicts with:
  'Packages/com.google.firebase.app/Firebase/Plugins/x86_64/FirebaseCppAppCheck.dll.meta'
We can't assign a new GUID because the asset is in an immutable folder. The asset will be ignored.
GUID [97216acd9a6147259ea13b994ccb9fa2] for asset 'Packages/com.google.firebase.app-check/Firebase/Plugins/x86_64/FirebaseCppAppCheck.so.meta' conflicts with:
  'Packages/com.google.firebase.app/Firebase/Plugins/x86_64/FirebaseCppAppCheck.so.meta'
We can't assign a new GUID because the asset is in an immutable folder. The asset will be ignored.
GUID [e67484c6c4644e6ab030360a71bfa7da] for asset 'Packages/com.google.firebase.app/Firebase/Plugins/x86_64/FirebaseCppAppCheck.bundle.meta' conflicts with:
  'Packages/com.google.firebase.app-check/Firebase/Plugins/x86_64/FirebaseCppAppCheck.bundle.meta'
We can't assign a new GUID because the asset is in an immutable folder. The asset will be ignored.
GUID [b45f6eb22b224bd4a0a45d3ddc8b9c7d] for asset 'Packages/com.google.firebase.app/Firebase/Plugins/x86_64/FirebaseCppAppCheck.dll.meta' conflicts with:
  'Packages/com.google.firebase.app-check/Firebase/Plugins/x86_64/FirebaseCppAppCheck.dll.meta'
We can't assign a new GUID because the asset is in an immutable folder. The asset will be ignored.
GUID [97216acd9a6147259ea13b994ccb9fa2] for asset 'Packages/com.google.firebase.app/Firebase/Plugins/x86_64/FirebaseCppAppCheck.so.meta' conflicts with:
  'Packages/com.google.firebase.app-check/Firebase/Plugins/x86_64/FirebaseCppAppCheck.so.meta'
We can't assign a new GUID because the asset is in an immutable folder. The asset will be ignored.

This is because the firebase.core package includes some of the app-check binaries. So they are duplicated when using both firebase.app and firebase.app-check.

You can also just download the latest firebase.app .tgz file and check in the "Firebase/Plugins/x86_64/" directory. The AppCheck binaries should not be here.

@google-oss-bot
Copy link

I found a few problems with this issue:

  • I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight.
  • This issue does not seem to follow the issue template. Make sure you provide all the required information.

@AlmostMatt
Copy link
Contributor

AlmostMatt commented May 15, 2023

Thanks for the report. I can confirm that the app tgz is unintentionally including the appcheck binaries.

I'll followup internally to ensure that this is resolved. In the meantime you can try to delete the redundant appcheck files, or maybe import appcheck as a unitypackage instead of importing appcheck and app separately.

@nvandessel
Copy link

Also encountering this issue as I attempt to implement AppCheck. Personally, using tarballs is greatly preferred over importing Unity packages as it doesn't clutter the project. Deleting the redundant files does seem to resolve the issue, however will monitor here for update on fix. Thanks!

@AlmostMatt
Copy link
Contributor

This is resolved by #729, which will be in the next release (11.1.0).

@robinryf
Copy link
Author

Awesome 👏 Thank you for the quick work!
I also greatly prefer using the tarballs on our private NPM registry.

@a-maurice
Copy link
Collaborator

FYI, the 11.1.0 release is out now, which has this fix. https://github.com/firebase/firebase-unity-sdk/releases/tag/v11.1.0

@firebase firebase locked and limited conversation to collaborators Jun 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants