Skip to content

Comments

Change Firestore credentials to use shared_ptr#1827

Open
a-maurice wants to merge 5 commits intomainfrom
am-firestore_fix
Open

Change Firestore credentials to use shared_ptr#1827
a-maurice wants to merge 5 commits intomainfrom
am-firestore_fix

Conversation

@a-maurice
Copy link
Contributor

Description

Provide details of the change, and generalize the change in the PR title above.

Change Firestore's Auth and App Check credentials to use shared_ptr's instead of unique_ptr. This is because the iOS repo changed to use shared_ptrs recently, and the mismatch seems to be causing runtime errors. See firebase/firebase-ios-sdk#15558

Fixes firebase/firebase-unity-sdk#1383


Testing

Describe how you've tested these changes. Link any manually triggered Integration tests or CPP binary SDK Packaging Github Action workflows, if applicable.

Running tests locally


Type of Change

Place an x the applicable box:

  • Bug fix. Add the issue # below if applicable.
  • New feature. A non-breaking change which adds functionality.
  • Other, such as a build process or documentation change.

Notes

  • Bug fixes and feature changes require an update to the Release Notes section of release_build_files/readme.md.
  • Read the contribution guidelines CONTRIBUTING.md.
  • Changes to the public API require an internal API review. If you'd like to help us make Firebase APIs better, please propose your change in a feature request so that we can discuss it together.

@gemini-code-assist
Copy link
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@a-maurice a-maurice added the tests-requested: quick Trigger a quick set of integration tests. label Feb 18, 2026
@github-actions github-actions bot added tests: in-progress This PR's integration tests are in progress. and removed tests-requested: quick Trigger a quick set of integration tests. labels Feb 18, 2026
@github-actions
Copy link

github-actions bot commented Feb 18, 2026

✅  Integration test succeeded!

Requested by @a-maurice on commit 4416844
Last updated: Wed Feb 18 12:24 PST 2026
View integration test log & download artifacts

@github-actions github-actions bot added the tests: failed This PR's integration tests failed. label Feb 18, 2026
@firebase-workflow-trigger firebase-workflow-trigger bot removed the tests: in-progress This PR's integration tests are in progress. label Feb 18, 2026
@a-maurice a-maurice added tests-requested: quick Trigger a quick set of integration tests. and removed tests: failed This PR's integration tests failed. labels Feb 18, 2026
@github-actions github-actions bot added tests: in-progress This PR's integration tests are in progress. tests: succeeded This PR's integration tests succeeded. and removed tests-requested: quick Trigger a quick set of integration tests. labels Feb 18, 2026
@firebase-workflow-trigger firebase-workflow-trigger bot removed the tests: in-progress This PR's integration tests are in progress. label Feb 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests: succeeded This PR's integration tests succeeded.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Firestore (bad_weak_ptr) crash on iOS 15.5 for Firestore SDK 13.6.0+

1 participant