-
Notifications
You must be signed in to change notification settings - Fork 2.3k
[Firebase Storage]: ref.getMetadata doesn't return property downloadURLs on iOS. returns relative url on Android #791
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
Comments
@escobar5 have you tried using the ref's |
@chrisbianca yes, but I need the updated metadata also, so it would be better to do just one request |
Ok, thanks for confirming - I've flagged it as a bug that we'll look at once we've got our messaging rewrite out of the way. |
Thanks for reporting this issue. We're aware that Storage has fallen behind slightly on React Native Firebase and would like to bring it up to speed again. I will close this issue for now and track it as well as other issues collectively over on the Storage improvements proposal to be addressed in a future release. See #1260 |
@escobar5 confused as to where you're reading that https://firebase.google.com/docs/reference/ios/firebasestorage/api/reference/Classes/FIRStorageMetadata#file Loving
|
I can't find it, maybe it was removed in some version, I looked into my code and I'm not using that property anymore. I'm constructing the URL like this: var uri = `https://firebasestorage.googleapis.com/v0/b/${metadata.bucket}/o/${encodeURIComponent(metadata.fullPath||metadata.name)}?alt=media`; |
@escobar5 ok I'll close this issue for now then. Thanks for your responses |
Uh oh!
There was an error while loading. Please reload this page.
Issue
On iOS the getMetadata() object doesn't have the property downloadURLs
Firebase docs
On Android it does return the property, but the url is relative (eg. /images/img.png)
I've tried getting the metadata using firebase's web sdk and it does returns the property and the url is absolute.
Environment
The text was updated successfully, but these errors were encountered: