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 typo in ENDPOINT_URL_ADDRESS variable #9

Merged
merged 2 commits into from
Nov 13, 2023

Conversation

firedigger
Copy link
Contributor

I have utilized the script on Windows, and I wasn't able to get s3cmd to work, but s4cmd worked nicely. I had to modify several places in the script to make it work. I don't believe the script is assumed to be used on windows and I am not good at bash, so I won't propose all fixes, but this PR contains a universal fix.
In addition to the typo fix s4cmd seems to require a slash in the URL as I got from bloomreach/s4cmd#93 (comment)

@christianbaun christianbaun merged commit 353a719 into christianbaun:master Nov 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants