Skip to content

Revert "url_exists related improvements (#34095)"#34144

Merged
haampie merged 1 commit intodevelopfrom
revert-34095-fix/url-things
Nov 26, 2022
Merged

Revert "url_exists related improvements (#34095)"#34144
haampie merged 1 commit intodevelopfrom
revert-34095-fix/url-things

Conversation

@balay
Copy link
Copy Markdown
Contributor

@balay balay commented Nov 26, 2022

This reverts commit d06fd26.

$ ./bin/spack fetch blaspp
==> Error: All fetchers failed for spack-stage-blaspp-2022.07.00-zrcutz3ecr5dikupw4paiqnzo5ht5i7r

@spackbot-app spackbot-app bot added core PR affects Spack core functionality fetching tests General test capability(ies) utilities labels Nov 26, 2022
@balay balay added the xSDK label Nov 26, 2022
@haampie
Copy link
Copy Markdown
Member

haampie commented Nov 26, 2022

For posterity: the problem is that Bitbucket's API forwards download requests to an S3 bucket using a temporary URL. This URL includes a signature for the request, which embeds the HTTP verb. That means only GET requests are allowed, and HEAD requests would fail verification, leading to 403 erros. The same is observed when using curl -LI ...

@haampie haampie enabled auto-merge (squash) November 26, 2022 17:36
@haampie haampie merged commit 12c23f2 into develop Nov 26, 2022
@haampie haampie deleted the revert-34095-fix/url-things branch November 26, 2022 17:56
amd-toolchain-support pushed a commit to amd-toolchain-support/spack that referenced this pull request Feb 16, 2023
This reverts commit d06fd26.

The problem is that Bitbucket's API forwards download requests to an S3 bucket using a temporary URL. This URL includes a signature for the request, which embeds the HTTP verb. That means only GET requests are allowed, and HEAD requests would fail verification, leading to 403 erros. The same is observed when using `curl -LI ...`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core PR affects Spack core functionality fetching tests General test capability(ies) utilities xSDK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants