Skip to content

Commit

Permalink
configure the browser extensions to point to your DOMAIN:8443
Browse files Browse the repository at this point in the history
  • Loading branch information
@tpizah
tpizah committed Aug 14, 2015
1 parent 0d209fd commit 542f2dd
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions centos/docker-entrypoint.sh
View file
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,10 @@ openssl pkcs12 -export -inkey server.key -in server.crt -name mitro_server -out
cp server.jks /srv/mitro/mitro-core/build/java/src/co/mitro/core/server/debug_keystore.jks
cp server.jks /srv/mitro/mitro-core/java/server/src/co/mitro/core/server/debug_keystore.jks

# configure the browser extensions
sed -i "s/www.mitro.co\|mitroaccess.com\|secondary.mitro.ca/${DOMAIN}/" /srv/mitro/browser-ext/login/common/config/config.release.js
sed -i "s/443/8443/" /srv/mitro/browser-ext/login/common/config/config.release.js

# exec command
cd /srv/mitro/mitro-core
exec "$@"

3 comments on commit 542f2dd

@cvladan
Copy link

@cvladan cvladan commented on 542f2dd Sep 9, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a small bug in line 57 (sed -i "s/443/8443/" ....) as it keeps prefixing with 8443, 88443, 888443, when doing docker start / stop. Just change that line to "s/= 443/= 8443/" and it should be ok.

It would also be nice that port number be in environment variable. Just maybe.

@servomac
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the report, I will try to fix it ASAP, also adding an environment variable to specify the port.

@servomac
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cvladan it's fixed at commit eea02e6

You can also define your own port using MITRO_PORT. Sorry for the delay solving this small bug :p

Please sign in to comment.