We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Dependencies
"react": "16.13.1", "react-native": "0.63.4", "rn-fetch-blob": "^0.12.0"
Problem
Network task fails to complete in Release mode (iOS) and seem to be blocking here, where self.task becomes nil and [nil resume] is called.
self.task
nil
[nil resume]
... self.task = [session dataTaskWithRequest:req]; [self.task resume]; // self.task = nil
Workaround
Avoiding self.task makes the network requests work again, so we suspect a problem around it.
// self.task = [session dataTaskWithRequest:req]; // [self.task resume]; [[session dataTaskWithRequest:req] resume];
Proposal Kindly select an option that works for both of us so a PR can be submitted:
weak
strong
NSURLSessionDataTask *task = [session dataTaskWithRequest:req]; [task resume]; self.task = task;
Work will be started immediately after response, because this issue is critical to us.
Edit: Submitted option 3 as a PR, cause it looks the least intrusive and would really help.
The text was updated successfully, but these errors were encountered:
92ce51a
No branches or pull requests
Dependencies
Problem
Network task fails to complete in Release mode (iOS) and seem to be blocking here, where
self.task
becomesnil
and[nil resume]
is called.Workaround
Avoiding
self.task
makes the network requests work again, so we suspect a problem around it.Proposal
Kindly select an option that works for both of us so a PR can be submitted:
self.task
property altogether. (probably not possible because it's used on cancelRequest)weak
? Declareself.task
asstrong
.Work will be started immediately after response, because this issue is critical to us.
Edit: Submitted option 3 as a PR, cause it looks the least intrusive and would really help.
The text was updated successfully, but these errors were encountered: