Many links broken #1838

New Issue

GiteaMirror · 2025-11-02T04:14:45-06:00

GiteaMirror commented

2025-11-02 04:14:45 -06:00

Originally created by @ghost on GitHub (May 30, 2018).

Gitea version (or commit ref): 15f6ec9
Git version: N/A
Operating system: N/A
Database (use [x]): N/A
- PostgreSQL
- MySQL
- MSSQL
- SQLite
Can you reproduce the bug at https://try.gitea.io:
- Yes (provide example URL)
  - https://try.gitea.io/yasuokav/link-test/issues/2
  - https://try.gitea.io/yasuokav/link-test/src/branch/master/README.md
- No
- Not relevant
Log gist: N/A

Description

The link should be
https://try.gitea.io/yasuokav/link-test/issues/1
instead of
https://try.gitea.io/yasuokav/link-test/issues/yasuokav/link-test/issues/1

Please check the links above for more details.

I think this issue because of #4058 @JonasFranzDEV

Screenshots

N/A

Originally created by @ghost on GitHub (May 30, 2018). - Gitea version (or commit ref): 15f6ec9 - Git version: N/A - Operating system: N/A - Database (use `[x]`): N/A - [ ] PostgreSQL - [ ] MySQL - [ ] MSSQL - [ ] SQLite - Can you reproduce the bug at https://try.gitea.io: - [x] Yes (provide example URL) - https://try.gitea.io/yasuokav/link-test/issues/2 - https://try.gitea.io/yasuokav/link-test/src/branch/master/README.md - [ ] No - [ ] Not relevant - Log gist: N/A ## Description The link should be `https://try.gitea.io/yasuokav/link-test/issues/1` instead of `https://try.gitea.io/yasuokav/link-test/issues/yasuokav/link-test/issues/1` Please check the links above for more details. I think this issue because of #4058 @JonasFranzDEV ## Screenshots N/A

GiteaMirror added the type/bug label 2025-11-02 04:14:45 -06:00

GiteaMirror closed this issue

2025-11-02 04:14:45 -06:00

GiteaMirror commented

2025-11-02 04:14:46 -06:00

@sapk commented on GitHub (May 30, 2018):

It should be related to the remove of leading '/' when base isn't absolute (with a scheme). This test should be change to add a check if base start with a '/' to keep it.
As this error is related to a suggested solution of mine ^^.

@sapk commented on GitHub (May 30, 2018): It should be related to the remove of leading '/' when base isn't absolute (with a scheme). This test should be change to add a check if base start with a '/' to keep it. As this error is related to a suggested solution of mine ^^.

GiteaMirror commented

2025-11-02 04:14:46 -06:00

@lunny commented on GitHub (May 30, 2018):

@JonasFranzDEV And I'm afraid release/v1.4 is also affected.

@lunny commented on GitHub (May 30, 2018): @JonasFranzDEV And I'm afraid release/v1.4 is also affected.

GiteaMirror commented

2025-11-02 04:14:46 -06:00

@sapk commented on GitHub (May 30, 2018):

@lunny fix is ready. I will backport it.

@sapk commented on GitHub (May 30, 2018): @lunny fix is ready. I will backport it.

GiteaMirror referenced this issue

2025-11-02 12:01:47 -06:00