Skip to content
Permalink
Browse files Browse the repository at this point in the history
chore(admin) restrict the admin port to 127 (#350)
* fix(admin) bind admin port to 127 only

* tests need to adjust the compose file for swarm vs docker-compose
  • Loading branch information
hutchic committed Mar 20, 2020
1 parent e94b5a4 commit dfa095c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
4 changes: 2 additions & 2 deletions compose/docker-compose.yml
Expand Up @@ -71,9 +71,9 @@ services:
- kong-net
ports:
- "8000:8000/tcp"
- "8001:8001/tcp"
- "127.0.0.1:8001:8001/tcp"
- "8443:8443/tcp"
- "8444:8444/tcp"
- "127.0.0.1:8444:8444/tcp"
healthcheck:
test: ["CMD", "kong", "health"]
interval: 10s
Expand Down
3 changes: 2 additions & 1 deletion tests.sh
Expand Up @@ -29,7 +29,7 @@ done

sleep 20
curl -I localhost:8001 | grep 'Server: openresty'

sed -i -e 's/127.0.0.1://g' docker-compose.yml
KONG_DOCKER_TAG=${KONG_DOCKER_TAG} docker stack deploy -c docker-compose.yml kong
sleep 20
until docker ps | grep ${KONG_DOCKER_TAG}:latest | grep -q healthy; do
Expand All @@ -45,6 +45,7 @@ docker stack rm kong
sleep 20
docker swarm leave --force
docker volume prune -f
git checkout -- docker-compose.yml
popd

# Validate Kong is running as the Kong user
Expand Down

0 comments on commit dfa095c

Please sign in to comment.