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

Package links with plus character result in broken redirect #63

Closed
palmskog opened this issue Jan 18, 2025 · 4 comments
Closed

Package links with plus character result in broken redirect #63

palmskog opened this issue Jan 18, 2025 · 4 comments
Labels
bug Something isn't working packages About the Rocq Opam Packages

Comments

@palmskog
Copy link
Contributor

Generated links to packages with versions that have a + character in them give a redirect error. For example, Firefox gives the following error:

The page isn’t redirecting properly

Examples of package links that are broken like this:

@palmskog palmskog added bug Something isn't working packages About the Rocq Opam Packages labels Jan 18, 2025
@mattam82
Copy link
Member

After a cursory look I couldn't find the bug yet... Needs to be investigated some more. Probably also affects ocaml.org

@cuihtlauac
Copy link

ocaml.org doesn't seem to be affected:

https://ocaml.org/p/mccs/1.1%2B9
https://ocaml.org/p/libtorch/2.2.1%2Blinux-x86_64

Remember that ocaml.org doesn't manage to produce documentation for all opam packages & versions. This test has false negative results. Could it be the case on your side, too?

@Zimmi48
Copy link
Member

Zimmi48 commented Jan 25, 2025

Now, this bug has become more serious because it affects links for the Rocq 9.0+rc1 release as well:

The last one in particular is a link that is shown prominently on the front page.

@Zimmi48
Copy link
Member

Zimmi48 commented Jan 27, 2025

Apparently, it was an issue with the reverse proxy (outside the code of rocq-prover.org) which has been fixed: https://coq.zulipchat.com/#narrow/channel/476133-rocq-prover.2Eorg/topic/9.2E0.2Brc1.20links.20.2F.20display/near/495999454

@Zimmi48 Zimmi48 closed this as completed Jan 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working packages About the Rocq Opam Packages
Projects
None yet
Development

No branches or pull requests

4 participants