Skip to content

Commit

Permalink
fix: Change server address to accept all requests
Browse files Browse the repository at this point in the history
* ci: Fix container health check command
  • Loading branch information
mehmetumit committed Nov 6, 2023
1 parent d510efd commit ec9cffe
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ jobs:
run: |
docker run --name dexus -d -p 8080:8080 --rm dexus:latest
sleep 5
if [ ! "$(curl -fs localhost:8080/health)" ]; then
if ! curl -fs localhost:8080/health; then
printf 'err: container is not running properly' >&2
exit 1
fi
Expand Down
2 changes: 1 addition & 1 deletion cmd/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ func main() {
RedirectionRepo: redirectRepo,
})
rest.NewServer(logger, &rest.ServerConfig{
Addr: "localhost:8080",
Addr: ":8080",
//IdleTimeout: 90 * time.Second,
ReadTimeout: 1 * time.Second,
WriteTimeout: 1 * time.Second,
Expand Down

0 comments on commit ec9cffe

Please sign in to comment.