8000 Read ENDURAIN_HOST from config.json by jameswynn · Pull Request #214 · joaovitoriasilva/endurain · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Read ENDURAIN_HOST from config.json #214

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

Closed
wants to merge 1 commit into from

Conversation

jameswynn
Copy link

Addresses #213

This adds a config.json in the frontend/app/public dir which can store non-sensitive runtime configuration without the need for sed. I've used this approach several times in other applications, though not in Vue so the implementation may not be idiomatic.

@joaovitoriasilva
Copy link
Owner
joaovitoriasilva commented Jun 24, 2025

Ok, I understand. :)
Now I have a question. I have a lot of people reaching out because they say the setup of Endurain is not easy. In my opinion the problem is not Endurain itself, but it is related to the fact that people do not fully understand how Docker, Postgres, etc works.
Now adding a new file that a user needs to manipulate will add more complexity and more things for a user to do.

With that in mind, what do you recommend in order to keep this easier for these use cases?

EDIT: New volume, for example?

  • ./config.json:/frontend/app/public/config.json

@jameswynn
Copy link
Author

I didn't think it was very difficult to set up, but there were some parts that could be smoothed out. There are a lot of file mounts currently. Maybe you could have a generic "data" directory that includes all those subdirectories by default. I've seen many containers that just make this /data.

In kubernetes I mount the config.json from a configmap, so mounting it as a volume in docker would make a lot of sense as well. If you want to make it easier for the average user that doesn't care as much about immutability then the start.sh script could check for the config.json and create it from an environment variable if its missing.

@joaovitoriasilva
Copy link
Owner

Yes, the issue with the multiple volumes mount will be fixed in v0.13.0 :)
Regarding the immutable part I spent the last hour trying to achieve a solution that does not break the current logic. I created a new branch with my proposed solution. I would like your opinion on it if possible :)

And sorry for not pushing to this PR

@jameswynn
Copy link
Author

That's a very clean approach. I like it.

@joaovitoriasilva joaovitoriasilva linked an issue Jun 25, 2025 that may be closed by this pull request
@joaovitoriasilva
Copy link
Owner

Implemented on v0.12.4

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.

[FEATURE]: Non-Root, immutable container image
2 participants
0