Skip to content

Commit 3e33089

Browse files
committed
Fix verifying deployment when env.PORT is set
1 parent 78d569e commit 3e33089

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

src/plugins/meteor/assets/meteor-deploy-check.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ while [[ true ]]; do
5454

5555
# If the container restarted, the ip address would have changed
5656
# Get the current ip address right before it is used
57-
if [[ $HOST_NETWORK ]]; then
57+
if [[ $HOST_NETWORK == 0 ]]; then
5858
CONTAINER_IP="localhost"
5959
else
6060
CONTAINER_IP=$(docker inspect $APPNAME --format "{{.NetworkSettings.IPAddress}}")

src/plugins/meteor/utils.js

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,14 @@
11
export function checkAppStarted(list, api) {
22
const script = api.resolvePath(__dirname, 'assets/meteor-deploy-check.sh');
3-
const { app, proxy } = api.getConfig();
4-
const exposedPort = app.deployCheckPort || app.env.PORT || 80;
5-
const publishedPort = 80;
3+
const { app } = api.getConfig();
4+
const publishedPort = app.docker.imagePort || 80;
65

76
list.executeScript('Verifying Deployment', {
87
script,
98
vars: {
109
deployCheckWaitTime: app.deployCheckWaitTime || 60,
1110
appName: app.name,
12-
deployCheckPort: proxy ? publishedPort : exposedPort
11+
deployCheckPort: publishedPort
1312
}
1413
});
1514

0 commit comments

Comments
 (0)