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

Example to start Atomic Server with Cloudflared tunnel and 1password cli #752

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

AlexMikhalev
Copy link
Collaborator

PR Checklist:

Added documentation and sample of how to start the atomic server with docker-compose.

Signed-off-by: AlexMikhalev <alex@metacortex.engineer>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should probably not be included, right? Or do we need this .env in this docs folder?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here - I'd guess we should not include this file right?

```
TUNNEL_URL=http://atomic-server:8080
TUNNEL_TOKEN=op://at.terraphim.dev/token
ATOMIC_SERVER_URL=op://Shared/at.terraphim.dev/server_url
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was a little confused by this type of URL. Perhaps the OnePassword thing should be left out?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can move configs into sample folder - I don't mind including .env since it's one password URL to fetch data. I think it's easier to have fully functioning docker compose + env, but it depends on secrets, hence 1password thing.

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.

None yet

2 participants