[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1110429: marked as done (qa.debian.org: 404 Not found due to trailing slash breaks codesearch.d.o)

Your message dated Mon, 11 Aug 2025 23:58:12 +0200
with message-id <aJpndITJda1Fe2zr@brokenwa.re>
and subject line Re: Bug#1110429: qa.debian.org: 404 Not found due to trailing slash breaks codesearch.d.o
has caused the Debian Bug report #1110429,
regarding qa.debian.org: 404 Not found due to trailing slash breaks codesearch.d.o
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
1110429: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1110429
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: qa.debian.org
Severity: important
User: qa.debian.org@packages.debian.org
Usertags: debsources
X-Debbugs-Cc: tj.iam.tj@proton.me

Using codesearch.debian.net all results links to files are broken and
result in HTTP 404 errors from sources.debian.org due to the (new)
trailing slash suffix.

Commit 71297d80 appears to be due to the same issue but my brief code
inspection didn't identify why the trailing slash is now required.

I've also reported for dcs:

https://github.com/Debian/dcs/issues/127

Example:

https://codesearch.debian.net/search?q=hardware-in-the-loop&literal=1

https://codesearch.debian.net/show?file=crrcsim_0.9.13-3.2%2Fdocumentation%2Finput_method%2FMNAV%2Fautopilot.txt&line=77

resolves to:

https://sources.debian.org/src/crrcsim/0.9.13-3.2/documentation/input_method/MNAV/autopilot.txt?hl=77#L77

and results in a HTTP 404.

Whereas the new URL style with trailing slash requires:

https://sources.debian.org/src/crrcsim/0.9.13-3.2/documentation/input_method/MNAV/autopilot.txt/?hl=77#L77

--- End Message ---
--- Begin Message --- 
Hi Tj,

On Tue, Aug 05, 2025 at 10:23:46AM +0000, Tj wrote:
> Package: qa.debian.org
> Severity: important
> User: qa.debian.org@packages.debian.org
> Usertags: debsources
> X-Debbugs-Cc: tj.iam.tj@proton.me
> 
> Using codesearch.debian.net all results links to files are broken and
> result in HTTP 404 errors from sources.debian.org due to the (new)
> trailing slash suffix.
> 
> Commit 71297d80 appears to be due to the same issue but my brief code
> inspection didn't identify why the trailing slash is now required.
> 
> I've also reported for dcs:
> 
> https://github.com/Debian/dcs/issues/127
> 
> Example:
> 
> https://codesearch.debian.net/search?q=hardware-in-the-loop&literal=1
> 
> https://codesearch.debian.net/show?file=crrcsim_0.9.13-3.2%2Fdocumentation%2Finput_method%2FMNAV%2Fautopilot.txt&line=77
> 
> resolves to:
> 
> https://sources.debian.org/src/crrcsim/0.9.13-3.2/documentation/input_method/MNAV/autopilot.txt?hl=77#L77
> 
> and results in a HTTP 404.
> 
> Whereas the new URL style with trailing slash requires:
> 
> https://sources.debian.org/src/crrcsim/0.9.13-3.2/documentation/input_method/MNAV/autopilot.txt/?hl=77#L77

Thanks for reporting, and apologies in the delay getting back to this
important issue.

It has been fixed in commit 80641b790c2da32f48894f88e70b265731937d81
and deployed to sources.debian.org. Let me know if you experience any
other unexpected 404.

Best,
--
Matthieu

Attachment: signature.asc
Description: PGP signature


--- End Message ---
Reply to: