-
Notifications
You must be signed in to change notification settings - Fork 65
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
support alternate download endpoint urls #48
support alternate download endpoint urls #48
Conversation
It'd also be useful to provide the replacement regex. I have the same need as what you're saying, but my reverse proxy chops off more of the URL ( e.g. https://S3/bucket-name/asdf/* -> https://mysite.com/*) |
adds ability to control the regex for replacing s3 url with download_endpoint
For what it counts, I have pulled this fork into my project and it works flawlessly. Thanks!! |
@dlethin merge conflicts - maybe sync with upstream and refresh the PR ? 🙏 |
PR synced with upstream. |
Any chance of this being merged? I would absolutely love this functionality due to my S3 bucket being hosted behind Cloudfront and not being publicly accessible. |
I also need this feature as I've different public URLs for objects uploaded. |
I also have a need for exactly this feature - Cloudfront seems to be the preferred way to share releases on S3 as it makes it a lot easier to add security features to control access to the assets. Our configuration means that the files are not actually accessible from the direct S3 URL at all. I forked this branch and updated with the latest commits from the upstream master, to verify: Another confirmation on my end that the updated code works perfectly in my tests of a live app that we’re releasing. @joshdholtz is there anything I can do to help get this merged? Happy to update the README with a better example, also, if that's blocking in any way. |
I will take a look at this today! Just need to get a cup of ☕️ in my first 😊 |
Hey @joshdholtz - you are probably super busy with much more important things than this PR - just checking in to see whether there's any likelihood of this being merged. I'll go down the hacky route if not, but would love to see this "natively" in fastlane_s3 if possible 😄 |
Let me get some coffee in me and then I’ll take a look at this PR! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great to me! Sorry for slow reply on this. No excuses but I have a my life sorted out now so I will be much faster going forward 😊
🙌 Always appreciated. Thanks for all the time you put into this stuff. |
- 📚 add documentation for fastlane-community#48
This fix is meant to address #47