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

Fix ERROR: You potentially provided an mal-formed URL! #605

Closed
wants to merge 2 commits into from
Closed

Fix ERROR: You potentially provided an mal-formed URL! #605

wants to merge 2 commits into from

Conversation

djallalzoldik
Copy link

Hi, when you have a URL like this
http://vimeo.com/api/oembed.json?url=/web/20131219172814/http://vimeo.com/80375746&api=1&player_id=preroll&height=354&width=630&callback

this will cause an ERROR
which is ERROR: You potentially provided an mal-formed URL!

So as you can see the URL is not mal-formed
the code check if there are two Colon, and try to extract the port after the second colon in the URL, but the after the second colon there are string not digit so that will cause the ERROR

So I add some lines to check if the value after the second colon is a digit or not

Hi when we have a URL like this 
http://vimeo.com/api/oembed.json?url=/web/20131219172814/http://vimeo.com/80375746&api=1&player_id=preroll&height=354&width=630&callback
will cause an Error that is 
You potentially provided a malformed URL!

So as you can see the URL is not mal-formed, so I add 3 lines to fix that issue 
which checks if the character after the ":" is a digit
@djallalzoldik djallalzoldik closed this by deleting the head repository Feb 22, 2023
@Talanor Talanor mentioned this pull request May 29, 2024
Relkci added a commit to Relkci/EyeWitness that referenced this pull request Jun 3, 2024
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.

1 participant