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.

Sign up for GitHub

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

Support non-root #52

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
+2 −2
Open

Support non-root #52

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
FEATURE: support non-root 
binding on port 80 needs root or extended capabilities. changing to a higher port supports non-root execution.
@bpereto
bpereto committed Feb 14, 2022

Verified

This commit was signed with the committer’s verified signature.
bpereto Bäm
GPG key ID: 73EB5662814EE3DE
Verified
Learn about vigilant mode
commit e63b797628c517d7c6f6398d890b7a23f16fb893
2 changes: 1 addition & 1 deletion Dockerfile
View file
Original file line number Diff line number Diff line change
@@ -23,4 +23,4 @@ COPY --from=builder /etc/ssl/certs/ca-certificates.crt /etc/ssl/certs/
COPY --from=builder /go/whoami/whoami .

ENTRYPOINT ["/whoami"]
EXPOSE 80
EXPOSE 8000
2 changes: 1 addition & 1 deletion app.go
View file
Original file line number Diff line number Diff line change
@@ -45,7 +45,7 @@ func init() {
flag.StringVar(&cert, "cert", "", "give me a certificate")
flag.StringVar(&key, "key", "", "give me a key")
flag.StringVar(&ca, "cacert", "", "give me a CA chain, enforces mutual TLS")
flag.StringVar(&port, "port", getEnv("WHOAMI_PORT_NUMBER", "80"), "give me a port number")
flag.StringVar(&port, "port", getEnv("WHOAMI_PORT_NUMBER", "8000"), "give me a port number")
flag.StringVar(&name, "name", os.Getenv("WHOAMI_NAME"), "give me a name")
}