Skip to content

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

Merged
acelaya merged 2 commits into
shlinkio:developfrom
matiasgarciaisaia:feature/server-from-env
May 29, 2021
Merged

Single-server servers.json from environment variables in Docker image#433
acelaya merged 2 commits into
shlinkio:developfrom
matiasgarciaisaia:feature/server-from-env

Conversation

@matiasgarciaisaia

Copy link
Copy Markdown
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
Copy Markdown

@codecov-commenter

codecov-commenter commented May 29, 2021

Copy link
Copy Markdown

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

acelaya commented May 29, 2021

Copy link
Copy Markdown
Member

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
Copy Markdown
Contributor Author

Awesome! :)

Thanks for updating the docs, too!

matiasgarciaisaia added a commit to matiasgarciaisaia/shlink-web-client that referenced this pull request May 29, 2021
Alfar0nt pushed a commit to Alfar0nt/dhiarlink-web-client that referenced this pull request Jun 15, 2026
…-from-env

Single-server servers.json from environment variables in Docker image
Alfar0nt pushed a commit to Alfar0nt/dhiarlink-web-client that referenced this pull request Jun 15, 2026
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