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

Stop videos from being downloaded as images #551

Merged
merged 4 commits into from
Nov 16, 2021
Merged

Stop videos from being downloaded as images #551

merged 4 commits into from
Nov 16, 2021

Conversation

OMEGARAZER
Copy link
Contributor

Erroneous .gifv extensions such as .giff .gifff or .gift resolve to a static image and are downloaded by the direct downloader. (ex: https://i.imgur.com/OGeVuAe.giff )

Copy link
Owner

@Serene-Arc Serene-Arc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, thanks for contributing :) For this could you please add tests showing that these extensions are now caught and resolve to the specific resource i.e. add tests for urls that would fail now and would succeed after this change.

@OMEGARAZER
Copy link
Contributor Author

I've added to the test cases, if I missed anything please let me know.

OMEGARAZER and others added 4 commits November 15, 2021 11:00
Erroneous .gifv extensions such as .giff or .gift resolve to a static image and are downloaded by the direct downloader. (ex: https://i.imgur.com/OGeVuAe.giff  )
Adding test for .giff/.gift imgur extension
@aliparlakci aliparlakci changed the base branch from master to development November 16, 2021 14:05
@aliparlakci aliparlakci merged commit 53562f4 into Serene-Arc:development Nov 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants