tangled.org / core
Monorepo for Tangled — https://tangled.org
fork atom

docker: fix config #198

closed
opened by knotbin.com targeting master from [deleted fork]: fix-docker

(i be traumatized) fixes knotguard options, along with some more config improvements and fixes in docker compose and dockerfile

0
by knotbin.com 0 comments
expand 11 commits
4ad48896
docker: update to unified binary deployment
ba36e70d
port env var
ee4f1701
Update docker-compose.yml
73dfa54c
Revert "Update docker-compose.yml"
2df99324
Update docker-compose.yml
17b1f0b5
Update tangled_sshd.conf
0cccf0c3
Update tangled_sshd.conf
c2c66e3e
Update tangled_sshd.conf
7a082c7a
is this anything
6720f045
is THIS anything
4abb37fa
optional caddy
sign up or login to add to the discussion
1
by knotbin.com 0 comments
expand 1 commit
46f92372
docker: config fixes
sign up or login to add to the discussion
2
by knotbin.com 0 comments
expand 2 commits
46f92372
docker: config fixes
5e2626ee
fix stupid stuff
sign up or login to add to the discussion
3
by knotbin.com 0 comments
expand 3 commits
46f92372
docker: config fixes
5e2626ee
fix stupid stuff
3c6c05b4
Update docker-compose.yml
sign up or login to add to the discussion
4
by knotbin.com 2 comments
expand 4 commits
46f92372
docker: config fixes
5e2626ee
fix stupid stuff
3c6c05b4
Update docker-compose.yml
d405c7c8
fix authkeyscommand
anirudh.fi

Thanks, looks a lot better:

  • the chown in the ENTRYPOINT seems like the wrong place for it; why not as a RUN directive in the dockerfile?
  • the AuthorizedKeysCommand, given the use of /bin/sh, perhaps should be a wrapper script after all? Just for the sake of cleanliness.
anirudh.fi

Additionally, perhaps a better commit message that's a bit more descriptive than "docker: fix config" is in order.

sign up or login to add to the discussion
5
by knotbin.com 0 comments
expand 1 commit
66bc7a08
docker: config fixes
sign up or login to add to the discussion
6
by knotbin.com 0 comments
expand 1 commit
6c43f34f
docker: fix authorized-keys & add config options
sign up or login to add to the discussion
7
by knotbin.com 0 comments
expand 1 commit
63c4cd76
docker: fix authorized-keys & add config options
sign up or login to add to the discussion
8
by knotbin.com 0 comments
expand 1 commit
657f5ac2
docker: fix authorized-keys & add config options
sign up or login to add to the discussion
9
by knotbin.com 1 comment
expand 1 commit
f058fe8f
docker: fix authorized-keys & add config options
anirudh.fi

Thanks for all the effort, Roscoe! This is now merged into the shiny new @tangled.sh/knot-docker repo. I'll close this now!

closed without merging
sign up or login to add to the discussion
Labels

None yet.

assignee

None yet.

Participants 2
AT URI
at://did:plc:6hbqm2oftpotwuw7gvvrui3i/sh.tangled.repo.pull/3lqjrzdoo7f22