Skip to content
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

[🐛] App crashed after select Video from Library in iOS 13 #1742

Closed
vivianlys90 opened this issue Jun 8, 2021 · 8 comments · Fixed by #1904
Closed

[🐛] App crashed after select Video from Library in iOS 13 #1742

vivianlys90 opened this issue Jun 8, 2021 · 8 comments · Fixed by #1904

Comments

@vivianlys90
Copy link

vivianlys90 commented Jun 8, 2021

Description

Getting this error after select Video from Library in iOS 13. App crashed at function didFinishPickingMediaWithInfo.

"Error returned from daemon: Error Domain=com.apple.accounts Code=7 "(null)""

Screenshot 2021-06-08 at 10 29 11 AM

How to repeat issue and example

  • Use this options for Video selection:
    {
    "durationLimit": 600,
    "maxHeight": 1000,
    "maxWidth": 1000,
    "mediaType": "video",
    "quality": 1,
    "videoQuality": "low"
    }
  • Choose Video from Library
  • Showing Video viewer screen
  • Tap on 'Choose' and compress video
  • App crashed

Additional Information

  • Image Picker version: 4.0.3
  • React Native version: 0.63
  • Platform: iOS
  • Development Operating System: MacOS
  • Dev tools: Xcode 12.4
vivianlys90 added a commit to tancy3318/react-native-image-picker that referenced this issue Jun 8, 2021
vivianlys90 added a commit to tancy3318/react-native-image-picker that referenced this issue Jun 9, 2021
vivianlys90 added a commit to tancy3318/react-native-image-picker that referenced this issue Jun 9, 2021
@m3rlin94
Copy link

Can confirm this is happening on v4.0.4

@vivianlys90
Copy link
Author

vivianlys90 commented Jun 29, 2021 via email

@m3rlin94
Copy link

Hi Naveen, I have submitted my fixes and now still waiting for code review.You can check this pull request #1743 Have a good day. Best Regards,Vivian Lee On Tuesday, 29 June 2021, 09:42:29 am MYT, Naveen Cooray @.***> wrote: Happening this to me as well. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

Thanks @vivianlys90! Will check that out

vivianlys90 added a commit to tancy3318/react-native-image-picker that referenced this issue Jun 29, 2021
@Muffles
Copy link

Muffles commented Jul 2, 2021

Same here, can confirm this is happening on 4.0.4

@klakshman8
Copy link

I am still having issue any solution

@vivianlys90
Copy link
Author

I have submitted PR for this issue, but still waiting for code merge. #1743

@musesfish
Copy link

when I picked a video in Android
APP crashed in recently
react–native 0.59.1
react–native–image–picker 0.28.1
so I set option mediaType:'photo'
but I still can choose video
I try to make the most simple option
and I find remove the option 'quality'
that can solve crash
but no 'quality' upload image is too slow
hope having a best solution

lukebrandonfarrell pushed a commit that referenced this issue Dec 28, 2021
…when selected (#1904)

* Fix GH-1742: Additional error check for nil around copying video and adds error message handling to prevent hard crash.
* Fix GH-1742: Lower case "nil" in conditional check
Johan-dutoit pushed a commit that referenced this issue Dec 28, 2021
## [4.6.1](v4.6.0...v4.6.1) (2021-12-28)

### Bug Fixes

* **videos:** videos not working on iOS 13 and throwing a fatal crash when selected ([#1904](#1904)) ([e45c862](e45c862)), closes [#1742](#1742) [#1742](#1742)
@Johan-dutoit
Copy link
Collaborator

🎉 This issue has been resolved in version 4.6.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

MattDennyir added a commit to MattDennyir/react-native-image-picker that referenced this issue Oct 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants