Skip to content

Single-server servers.json from environment variables in Docker image #433

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

Merged
merged 2 commits into from
May 29, 2021

Conversation

matiasgarciaisaia
Copy link
Contributor

In the Docker image, generate the servers.json file with a single server by reading environment variables.

This is useful to avoid mounts in hosted Docker environments.

In the Docker image, generate the servers.json with a single server
by reading environment variables.
@github-actions
Copy link

@codecov-commenter
Copy link

codecov-commenter commented May 29, 2021

Codecov Report

Merging #433 (f626f9b) into main (d5530b4) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #433   +/-   ##
=======================================
  Coverage   75.62%   75.62%           
=======================================
  Files         142      142           
  Lines        1670     1670           
  Branches      242      242           
=======================================
  Hits         1263     1263           
  Misses        213      213           
  Partials      194      194           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d5530b4...f626f9b. Read the comment docs.

@acelaya acelaya changed the base branch from main to develop May 29, 2021 09:32
@acelaya
Copy link
Member

acelaya commented May 29, 2021

Nice! I like the idea 🙂

I have just pushed a change renaming the env vars to be less redundant, becoming SHLINK_SERVER_URL, SHLINK_SERVER_API_KEY and SHLINK_SERVER_NAME.

@acelaya acelaya added this to the 3.2.0 milestone May 29, 2021
@acelaya acelaya merged commit 4c21ad0 into shlinkio:develop May 29, 2021
@matiasgarciaisaia
Copy link
Contributor Author

Awesome! :)

Thanks for updating the docs, too!

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.

3 participants