Skip to content

feat(index): resolve public artifact URL server-side to reduce redirects#8284

Merged
lotas merged 1 commit intotaskcluster:mainfrom
JohanLorenzo:fix-8035-single-redirect
Feb 18, 2026
Merged

feat(index): resolve public artifact URL server-side to reduce redirects#8284
lotas merged 1 commit intotaskcluster:mainfrom
JohanLorenzo:fix-8035-single-redirect

Conversation

@JohanLorenzo
Copy link
Contributor

@JohanLorenzo JohanLorenzo commented Feb 17, 2026

Github Issue: Fixes #8035

Depends on #8283. This PR just adds one commit, the last one. The other ones come from #8283. Patch also tested manually:

Before

curl -IL http://taskcluster/api/index/v1/task/e2e-test.bewit.artifacts/artifacts/public/hello.txt

HTTP/1.1 303 See Other
Content-Type: text/plain; charset=utf-8
Content-Length: 114
[...]
Location: http://taskcluster/api/queue/v1/task/TVtOlb97THWZYG36WoNxZg/artifacts/public%2Fhello.txt
[...]

HTTP/1.1 303 See Other
Content-Type: application/json; charset=utf-8
Content-Length: 112
[...]
location: http://taskcluster/public-bucket/TVtOlb97THWZYG36WoNxZg/0/public/hello.txt
[...]

HTTP/1.1 200 OK
Content-Type: text/plain; charset=utf-8
Content-Length: 6
[...]
Last-Modified: Mon, 16 Feb 2026 17:35:55 GMT
ETag: "79790eaf1bb29e4a543a90d69d8dbd9b"
[...]

After

HTTP/1.1 303 See Other
Content-Type: text/plain; charset=utf-8
Content-Length: 100
[...]
Location: http://taskcluster/public-bucket/TVtOlb97THWZYG36WoNxZg/0/public/hello.txt
[...]

HTTP/1.1 200 OK
Date: Tue, 17 Feb 2026 10:34:35 GMT
Content-Type: text/plain; charset=utf-8
Content-Length: 6
[...]
Last-Modified: Mon, 16 Feb 2026 17:35:55 GMT
ETag: "79790eaf1bb29e4a543a90d69d8dbd9b"
[...]

@JohanLorenzo JohanLorenzo changed the title Fix 8035 single redirect feat(index): resolve public artifact URL server-side to reduce redirects Feb 17, 2026
@JohanLorenzo JohanLorenzo mentioned this pull request Feb 17, 2026
Merged
@JohanLorenzo JohanLorenzo marked this pull request as ready for review February 17, 2026 11:06
@JohanLorenzo JohanLorenzo requested a review from a team as a code owner February 17, 2026 11:06
@JohanLorenzo JohanLorenzo requested review from lotas, matt-boris and petemoore and removed request for a team February 17, 2026 11:06
@JohanLorenzo
Copy link
Contributor Author

JohanLorenzo commented Feb 17, 2026

I genuinely don't know if moving this redirect on the server side is going to cut the extra 200ms seen in #8035. @lotas is there a way I can test it on a deployed instance? Using localhost doesn't enable me to test it.

@lotas
Copy link
Contributor

lotas commented Feb 17, 2026

I genuinely don't know if moving this redirect on the server side is going to cut the extra 200ms seen in #8035. @lotas is there a way I can test it on a deployed instance? Using localhost doesn't enable me to test it.

200ms depends on the region, as julien mentioned before for north america those are much lower, because storage is located in the same region as services

you can try setting TASKCLUSTER_ROOT_URL for index service to point to fxci (but you need proper credentials with correct permissions)

@JohanLorenzo
Copy link
Contributor Author

JohanLorenzo commented Feb 17, 2026

Discussed over Slack. Setting TASKCLUSTER_ROOT_URL locally won't likely show whether the fix works. In fact, I'm based in Europe and our Taskcluster instances are in North America. Hence, resolving the direct artifact URL on the server side isn't going to show any benefits because my server is in Europe anyway.

We agreed on merging this PR and testing on the dev instance.

@JohanLorenzo JohanLorenzo force-pushed the fix-8035-single-redirect branch from 5bb44e7 to 24df1e8 Compare February 18, 2026 10:38
@lotas lotas merged commit 99d7657 into taskcluster:main Feb 18, 2026
73 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lotas lotas lotas approved these changes

@petemoore petemoore Awaiting requested review from petemoore petemoore is a code owner automatically assigned from taskcluster/core

@matt-boris matt-boris Awaiting requested review from matt-boris matt-boris is a code owner automatically assigned from taskcluster/core

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Urls pointing to indexes redirect twice

2 participants

@JohanLorenzo @lotas