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

Can't use "record video" as upload option on Safari iOS #1568

Open
1 task done
ekzyis opened this issue Nov 10, 2024 · 5 comments
Open
1 task done

Can't use "record video" as upload option on Safari iOS #1568

ekzyis opened this issue Nov 10, 2024 · 5 comments

Comments

@ekzyis
Copy link
Member

ekzyis commented Nov 10, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Describe the bug

If you click on Upload, you can upload an existing file or take a photo or video.

If you choose video, the resulting video can't be uploaded because it uses .mov.

Screenshots

signal-2024-11-10-162656

Steps To Reproduce

  1. Click on "upload file"
  2. Select "Take Photo or Video"
  3. Record video and try to upload

Expected behavior

We support MOV format

Logs

No response

Device information

  • OS: iOS
  • Browser: Safari

Additional context

No response

@Soxasora
Copy link
Contributor

I was tinkering with this issue and allowing video/quicktime mime type fixes it.
The real problem is that embedded mov files can only be seen by Apple users per MDN. Transcoding to mp4 during the upload process could be an option though.

@huumn
Copy link
Member

huumn commented Nov 14, 2024

I was looking for something like imgproxy for video awhile ago but I couldn't find anything.

Why aren't more people making stuff we need 😅

@Soxasora
Copy link
Contributor

I could try to make a proof of concept with ffmpeg, this way not only SN could support mov files but also handle compression for all the other videos 🗜️

@ekzyis
Copy link
Member Author

ekzyis commented Nov 15, 2024

sounds good to me!

@Soxasora
Copy link
Contributor

Soxasora commented Nov 20, 2024

trim.D8C45A99-6AE4-4A67-B270-E92A7446654B.MOV

Transcoding 🔁
Being a proof-of-concept for now I'm using Ffmpeg and Node in a docker container.

It even adds an onTranscoding call to update the user on what's happening! (excited for so little)

Oh and hey mov files now work :P

EDIT: I also noticed a bug that prevents iOS (or Safari?) users from uploading videos, I'll open an issue and then PR the hotfix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants