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

HEAD (or other approach) links to fetch Content-Type for video/image preview #78

Open
chinese-soup opened this issue Nov 23, 2020 · 0 comments

Comments

@chinese-soup
Copy link
Collaborator

chinese-soup commented Nov 23, 2020

  • ... and therefore determine whether to show the Preview button or Open the preview (if "open link directly" option=true)
  • Should probably be an option, also, since some people don't like giving out their IP to a server without actually opening the link.

Current state:

This could also be used for a limit of Content-Size that is allowed to be (which could be an option) previewed in the app. (Since .e.g a 500 MB video file might not be a good idea to open in Qt? (just guessing, dunno, really))

Cons of HEAD:

  • Slower
  • HEAD can be disabled/unsupported on the destination server, fallback to endswith(".jpeg|.jpg") etc. then

Any other ideas?

@chinese-soup chinese-soup changed the title HEAD (or other approach) the links to see content-type for video/image HEAD (or other approach) links to fetch Content-Type for video/image preview Nov 23, 2020
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

1 participant