Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[fix] include custom url's "path" when creating Archive URL #234

Merged
merged 4 commits into from
Aug 1, 2024

Conversation

james0209
Copy link
Contributor

What

Currently when constructing the archive URL for download, only Scheme and Host are considered. This meams any Path within the custom URL is not used when constructing the Archive URL.

References

closes runatlantis/atlantis#4778

@james0209 james0209 requested a review from a team as a code owner July 24, 2024 18:41
@james0209
Copy link
Contributor Author

Hey @radeksimko, could we get a review/approval here please? 🙏

@james0209 james0209 changed the title [fix] Allow custom path as part of custom base URL [fix] include custom url's "path" when creating Archive URL Jul 24, 2024
@radeksimko radeksimko added the bug Something isn't working label Aug 1, 2024
Copy link
Member

@radeksimko radeksimko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That is reasonable patch, thanks for sending it in.

We naturally didn't run into this problem under tests as tests use the same path - just different scheme and host.

@radeksimko radeksimko merged commit b216d7f into hashicorp:main Aug 1, 2024
11 checks passed
@radeksimko
Copy link
Member

This was just released under v0.8.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

tf-download-url value seems to strip the path and only use the hostname.
2 participants