You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
... Or did you intend to call the completion helper with the files you got so far if an error occurs?
Since this seems to differ with the behaviour on Android (where the Task class will not allow you to reject and resolve the same task, I think?), I think adding a warning in the documentation could help anyway.
The text was updated successfully, but these errors were encountered:
@TPXP Thanks for the thorough report and analysis. I believe that your assumption is correct and a return should be added after the callback. We'll confirm with the Storage team and, if so, make the fix.
Step 0: Are you in the right place?
file a Github issue.
with the
firebase
tag.google group.
of the above categories, reach out to
Firebase Support.
this repository, please delete this section.
[REQUIRED] Step 1: Describe your environment
CocoaPods
[REQUIRED] Step 2: Describe the problem
Steps to reproduce:
listAllWithCompletion
on this StorageRefpermission denied
error, the other time with an empty directory result.Relevant Code:
Sorry, I suck at ObjC, here's a reproduction with the RNFirebase SDK invertase/react-native-firebase#4688
This issue can be summed up in one question: Is there a missing
return
here:firebase-ios-sdk/FirebaseStorage/Sources/FIRStorageReference.m
Line 417 in 14764b8
... Or did you intend to call the completion helper with the files you got so far if an error occurs?
Since this seems to differ with the behaviour on Android (where the
Task
class will not allow you to reject and resolve the same task, I think?), I think adding a warning in the documentation could help anyway.The text was updated successfully, but these errors were encountered: