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

some of url not working properly #139

Open
hareshlive opened this issue Apr 21, 2021 · 6 comments · May be fixed by #151
Open

some of url not working properly #139

hareshlive opened this issue Apr 21, 2021 · 6 comments · May be fixed by #151
Labels
good first issue This issue is good for beginners to start contributing. help wanted investigation-needed

Comments

@hareshlive
Copy link

www.cnn.com

https://www.cnn.com/2021/04/20/politics/maxine-waters-nancy-pelosi/index.html

this url no work

please help me about this issue

@LeonardoCardoso LeonardoCardoso added good first issue This issue is good for beginners to start contributing. help wanted investigation-needed labels Apr 24, 2021
@nikeshakya
Copy link

@hareshlive @LeonardoCardoso any update on this ? I am having similar issue with cnn links...Infinite waiting

@LeonardoCardoso
Copy link
Owner

@nikeshakya Hello! No updates. I haven’t had the time. Feel free to fork and investigate in the meantime.

@nikeshakya
Copy link

Hi @LeonardoCardoso,
because of following meta
" http-equiv="refresh" content="1;url=/2.265.2/static/unsupp.html" /"
it was running in infinite recursive function call
let me know if it helps in your debugging

@nikeshakya
Copy link

hi @LeonardoCardoso any updates please.... It has been so long...no response at all!

@LeonardoCardoso
Copy link
Owner

@nikeshakya you must understand that I have a life and other stuff to do. If it’s too urgent for you, feel free to fork the repo, investigate the issue and open a PR. I’m currently on vacations. Cheers!

@nikeshakya
Copy link

@LeonardoCardoso I was just asking you what's your thought on the result I gave you " http-equiv="refresh" content="1;url=/2.265.2/static/unsupp.html" /" basically CNN urls have tag like this

in such cases, refresh should be optional or should discard the line...

@nikeshakya nikeshakya linked a pull request Sep 9, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue This issue is good for beginners to start contributing. help wanted investigation-needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants