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

Firebase 11.x required for crashylitics fix. #174

Open
alessioborraccino opened this issue Oct 15, 2024 · 6 comments
Open

Firebase 11.x required for crashylitics fix. #174

alessioborraccino opened this issue Oct 15, 2024 · 6 comments
Labels
enhancement New feature or request

Comments

@alessioborraccino
Copy link

Describe the feature request?

Due to a recent issue with Firebase Crashylitics and Xcode 16, we would like to update Firebase to version 11.x, (since they promised a fix for this for next week), but the device authenticator is currently depending on OktaLogger, which doesn't support Firebase versions greater than 10, so our package resolution can't work.

Is there any plan to update the SDK so that we can update Firebase too?

Alessio from Autoscout24

New or Affected Resource(s)

Okta Logger would need an update

Provide a documentation link

No response

Additional Information?

No response

@alessioborraccino alessioborraccino added the enhancement New feature or request label Oct 15, 2024
@IldarAbdullin-okta
Copy link
Contributor

Hi @alessioborraccino ,

Push SDK depends on OktaLogger/FileLogger subspec in OktaLogger pod. So FileLogger doesn't have any dependencies on Firebase and Firebase update should not affect cocoapods integration

Please let me know if you have other questions

Ildar

@alessioborraccino
Copy link
Author

Sorry i should have mentioned it ealier, we are not using cocoapods (and can't), we are using spm, and Package.swift for oktalogger is defining a firebase dependency as

dependencies: [
        ...
        .package(url: "https://github.com/firebase/firebase-ios-sdk.git", .upToNextMajor(from: "10.3.0")),
]

as you can see is up to next major so therefore we have a conflict there.

@IldarAbdullin-okta
Copy link
Contributor

Thank you, @alessioborraccino

I think I understand where the issue is. Unfortunately SPM installs all transitive dependencies even if they are not required for the target. Our team will look into options how to fix this problem

Ildar

@alessioborraccino
Copy link
Author

Thanks for your response. Looking forward to a solution 🙏 🤞

@IldarAbdullin-okta
Copy link
Contributor

Hi @alessioborraccino

Can you try release 1.1.3 and let us know whether that fixes your issue with Firebase?

Ildar

@alessioborraccino
Copy link
Author

hi @IldarAbdullin-okta i just tested it and it seems to work fine :) thanks for your speed in handling this 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants