-
Notifications
You must be signed in to change notification settings - Fork 448
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
Fix video player crash #627
Comments
@ashitaprasad i am working on this issue |
Thanks @AffanShaikhsurab |
Interested to work on this, Can I work too on this issue @ashitaprasad ? |
as i am not able to reproduce the issue in windows can you please let me knowif you were using web / windows / android it would be a great help @ashitaprasad |
Thanks for investigating @AffanShaikhsurab |
@akshayw1 Do you use macOS? |
i have check on android and windows and there is no issue there !! looks like its an mac specific issue but need to be tested on linux !! |
Since I don't have access to macOS, I might be limited in resolving this issue effectively. it would be better for member with the appropriate setup to take this up! |
Yes, I use macOs air m2 |
@akshayw1 can you please check the logs and inspect what might have causing the issue ? |
Hi @ashitaprasad System: Sequoia macOS 15.3.1 PFA video demonstration: test.mp4Let me know if you need me to try any other scenario. Thanks! |
@ashitaprasad can you please specify what was the env that caused the issue ? |
@divyanshkul are you running it on the latest HEAD of the main branch? |
Updated the Issue description with all the details. |
@ashitaprasad Yes, I was running the latest HEAD till this 79788dc
|
Thanks for confirming @divyanshkul |
Hi @ashitaprasad , |
Hi @ashitaprasad, I have tested this issue on Windows, and I was able to reproduce it. Findings: Would love to discuss further and contribute the fix! Let me know if I should open a PR. 🚀 WhatsApp.Video.2025-03-05.at.11.26.08.PM.mp4 |
HI @ashitaprasad I am willing to work on this |
I was also looking at this issue .. I tried some fixes and then i noticed another bug.. switch to headers tab(Next to Response Body) then switch back to Response body.. this causes another crash.. I was testing it on linux Ubuntu 24.04.1 LTS. I think the widget is getting disposed then a update is being made to the widget But the widget is not in widget tree |
@ashitaprasad I identified the cause and fixed it . Will hit PR soon.. got some things to do Currently |
ok @badnikhil |
Describe the bug/problem
Launch API Dash
Paste the following URL in GET Request:
https://flutter.github.io/assets-for-api-docs/assets/videos/bee.mp4
Hit
Send
Once the response is received, the App crashes due to some error in VideoPreviewer - https://github.com/foss42/apidash/blob/main/lib/widgets/video_previewer.dart
This issue requires analysis of the root cause of the crash & potential fix.
Setup - Apple M1 (Sonoma 14.6.1)
The text was updated successfully, but these errors were encountered: