-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
🐛 [@react-native-firebase/functions] Missing stream
implementation for httpsCallable
functions
#8210
Comments
Neat, wasn't aware of that work. I believe we'll need underlying SDK support prior to us being able to do anything at this layer above them iOS: firebase/firebase-ios-sdk#14290 After that we'll have a fair bit of work here to wrap them and present them in a manner compatible with firebase-js-sdk (or we'd happily accept a PR of course...) The only thing I can think of in the meantime is that you can actually use the firebase-js-sdk alongside react-native-firebase if you like. It is bundled with react-native-firebase as it is used as a fallback for "other" platform support already internally here. You'll have to initialize an app manually after react-native-app boot just like you do on web, and if you use AppCheck you'll have to set the firebase-js-sdk functions headers with it after fetching it from react-native-firebase/app-check, similar with auth - but those headers shouldn't be too hard to set ? |
I looked into using |
Is a big chunk of work to support it here but most importantly we can't do anything not provided by the underling native android and iOS SDKs, so we are necessarily blocked on them regardless. Very nifty that it looks like it'll be possible to "weld together" the firebase-js-sdk library support for it while still using react-native-firebase for it's native-only stuff though |
Hello 👋, to help manage issues we automatically close stale issues. This issue has been automatically marked as stale because it has not had activity for quite some time.Has this issue been fixed, or does it still require attention?
Thank you for your contributions. |
Still requires attention |
@harveyappleton there's nothing we can do here, I just paid attention to it but it unfortunately wasn't attention well spent. You might look at the VertexAI implementation that we'll be landing soon though - it was able to use react-native fetch plus some polyfills to get readable stream support using javascript. That strategy can work: |
Hello 👋, to help manage issues we automatically close stale issues. This issue has been automatically marked as stale because it has not had activity for quite some time.Has this issue been fixed, or does it still require attention?
Thank you for your contributions. |
Streaming callable functions appears to have landed in firebase-android-sdk (33.11.0) and firebase-ios-sdk (11.10.0) now, this should be possible to implement now |
Hey amazing invertase team! Happy new year!
In Dec 2024, a PR was merged into
firebase-js-sdk
that allows you to stream responses from Firebase functions which is awesome! I've tried it out on web and it works wonderfully. Would be awesome if this could be implemented in RN Firebase Functions!!! :DThe text was updated successfully, but these errors were encountered: