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

https://pylint.pycqa.org/ keeps redirecting to non-HTTPS link #3802

Closed
alanyee opened this issue Aug 27, 2020 · 13 comments · Fixed by #4935
Closed

https://pylint.pycqa.org/ keeps redirecting to non-HTTPS link #3802

alanyee opened this issue Aug 27, 2020 · 13 comments · Fixed by #4935
Assignees
Labels
Maintenance Discussion or action around maintaining pylint or the dev workflow
Milestone

Comments

@alanyee
Copy link

alanyee commented Aug 27, 2020

I am unsure if this repo is the best place to raise this issue, but I have noticed that https://pylint.pycqa.org/ keeps redirecting http://pylint.pycqa.org/en/latest/ which is a HTTP link instead of the HTTPS link https://pylint.pycqa.org/en/latest/. I am unsure what other languages the pylint documentation maintains, but I have been experiencing the problem for English.

@AWhetter
Copy link
Contributor

AWhetter commented Sep 7, 2020

I'm not sure who looks after the domain so I've cross posted this to https://github.com/PyCQA/meta because the maintainers there may know more.

@graingert
Copy link
Contributor

The desired redirect pattern is:

http://pylint.pycqa.org -> 301 -> https://pylint.pycqa.org -> 302 -> https://pylint.pycqa.org/en/latest/

@graingert
Copy link
Contributor

@AWhetter it looks like this is an RTFD hosted page going via cloudflare?

Vary: Accept-Language, Cookie
X-RTD-Redirect: system
X-RTD-Domain: pylint.pycqa.org
X-RTD-Project: pylint
X-RTD-Project-Method: cname
X-RTD-Version-Method: path

might be possible to configure using an .readthedocs.yml

@graingert
Copy link
Contributor

graingert commented Sep 7, 2020

otherwise configurable with a meta-equiv-refresh redirect: https://github.com/sphinx-contrib/redirects/blob/master/sphinxcontrib/redirects/__init__.py#L18 or by @logilab and @PCManticore

@graingert
Copy link
Contributor

@AWhetter I'd highly recommend porting your rtfd config to .readthedocs.yml anyway - see https://github.com/PyCQA/modernize/blob/master/.readthedocs.yml

@graingert
Copy link
Contributor

@AWhetter
Copy link
Contributor

AWhetter commented Sep 7, 2020

Thanks for the help @graingert!
@PCManticore only you have permissions so I've assigned this to you, but feel free to give another maintainer permissions as well if you would prefer someone else to complete the work.

@graingert
Copy link
Contributor

@AWhetter you can use a meta-equiv-refresh and readthedocs.yml with only repo commit access

@PCManticore
Copy link
Contributor

@AWhetter @brycepg @hippo91 @Pierre-Sassoulas Let me know what is your readthedocs handle so I can add you to the project.

@Pierre-Sassoulas
Copy link
Member

@PCManticore, nice to see you :) My handle on read the doc is pierre-sassoulas. It's unrelated but could you also add me to the pypi group so I can publish a new version manually, please ?

@AWhetter
Copy link
Contributor

Mine is AWhetter

@hippo91
Copy link
Contributor

hippo91 commented Sep 15, 2020

@PCManticore happy to see you again! My handle is hippo91.

@DanielNoord
Copy link
Collaborator

This is still an issue. Have the readthedocs maintainers been updated? And if so, can this be fixed?

@Pierre-Sassoulas Pierre-Sassoulas added this to the 2.11.0 milestone Aug 31, 2021
@Pierre-Sassoulas Pierre-Sassoulas added Maintenance Discussion or action around maintaining pylint or the dev workflow and removed task labels Jun 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Maintenance Discussion or action around maintaining pylint or the dev workflow
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants