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

Adding S3 Endpoint Config Option #170

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

zacwallsie
Copy link

Allows for custom S3 endpoints to be added to the server config.

This pull request includes updates to the S3 configuration in the lib/controllers/share.js file and the serverconfig.json.example file to support custom S3 endpoints. These changes enhance the flexibility of the S3 service configuration.

Configuration updates:

  • lib/controllers/share.js: Added support for a custom S3 endpoint by updating the AWS configuration to include the endpoint option if provided in serviceOptions.
  • serverconfig.json.example: Added an optional endpoint field to allow users to specify a custom S3 endpoint in the configuration file.

zacwallsie and others added 2 commits November 7, 2024 15:54
Allows for custom S3 endpoints to be added to the server config.

Co-Authored-By: Michael Izatt <59045460+michaelizatt@users.noreply.github.com>
Altered implementation for easier readability.
@zacwallsie
Copy link
Author

Tested and Functional with Digital Ocean Spaces

lib/controllers/share.js Outdated Show resolved Hide resolved
@pjonsson
Copy link
Contributor

I can't test this PR right now, but I have local patches on TerriaMap to get the same effect, so it would be nice to have this PR merged.

Indentation converted to 4 spaces as apposed to 2 based on @pjonsson review.
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