Bad URL detection in Description field #1870

Closed
opened 2025-11-02 04:15:55 -06:00 by GiteaMirror · 8 comments
Owner

Originally created by @AloisMahdal on GitHub (Jun 5, 2018).

Description

If I include URL inside parentheses in Description of a git repo, gitea web UI will linkify it, but the closing bracket will be detected as art of the URL, resulting in HTTP-404 errors.

Screenshots

http://file.vornet.cz/tmp/rbrack.png

Originally created by @AloisMahdal on GitHub (Jun 5, 2018). - Gitea version (or commit ref): fa3ceb6 (try.gitea.io) - Git version: (not relevant) - Operating system: (unknown) - Database (use `[x]`): - [ ] PostgreSQL - [ ] MySQL - [ ] MSSQL - [ ] SQLite - Can you reproduce the bug at https://try.gitea.io: - [x] Yes - https://try.gitea.io/netvor/slop-copr-build - [ ] No - [ ] Not relevant - Log gist: ## Description If I include URL inside parentheses in Description of a git repo, gitea web UI will linkify it, but the closing bracket will be detected as art of the URL, resulting in HTTP-404 errors. ## Screenshots http://file.vornet.cz/tmp/rbrack.png
GiteaMirror added the issue/confirmedtype/bug labels 2025-11-02 04:15:55 -06:00
Author
Owner

@techknowlogick commented on GitHub (Jul 21, 2018):

I don't know if this is an issue that needs to be resolved, because there is already a URL field for URLs to go into.

@techknowlogick commented on GitHub (Jul 21, 2018): I don't know if this is an issue that needs to be resolved, because there is already a URL field for URLs to go into.
Author
Owner

@davidak commented on GitHub (Jul 21, 2018):

@techknowlogick if this usage is not intended, maybe don't detect URLs in this text at all?

@davidak commented on GitHub (Jul 21, 2018): @techknowlogick if this usage is not intended, maybe don't detect URLs in this text at all?
Author
Owner

@AloisMahdal commented on GitHub (Jul 25, 2018):

It could still make sense to include another URL in this field.

@AloisMahdal commented on GitHub (Jul 25, 2018): It could still make sense to include another URL in this field.
Author
Owner

@lunny commented on GitHub (Sep 7, 2018):

I like @davidak 's idea.

@lunny commented on GitHub (Sep 7, 2018): I like @davidak 's idea.
Author
Owner

@stale[bot] commented on GitHub (Jan 11, 2019):

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale[bot] commented on GitHub (Jan 11, 2019): This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.
Author
Owner

@stale[bot] commented on GitHub (Feb 21, 2019):

This issue has been automatically closed because of inactivity. You can re-open it if needed.

@stale[bot] commented on GitHub (Feb 21, 2019): This issue has been automatically closed because of inactivity. You can re-open it if needed.
Author
Owner

@davidak commented on GitHub (Feb 25, 2019):

I can still reproduce it on 28fcbf3: https://try.gitea.io/davidak/test4

@davidak commented on GitHub (Feb 25, 2019): I can still reproduce it on 28fcbf3: https://try.gitea.io/davidak/test4
Author
Owner

@mrsdizzie commented on GitHub (Mar 27, 2019):

This is fixed as of 71b734b (link in previous comment now works as expected)

@mrsdizzie commented on GitHub (Mar 27, 2019): This is fixed as of 71b734b (link in previous comment now works as expected)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#1870