Skip to content

Commit

Permalink
Move sed and fix typo in server/README.md
Browse files Browse the repository at this point in the history
Move the `sed` command below the definition of `JBOSS_HOME`.  It appears
standalone.xml doesn't exist at the point when the `sed` was run.
Additionally the environment variable `JBOSS_HOME` was not defined.

Also added the equals sign to the `server/README.md`. Without an equals
sign docker assumes true is the container name which causes it to fail.
  • Loading branch information
Dan Kelleher committed May 4, 2017
1 parent 52419be commit a8b1491
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 7 deletions.
10 changes: 4 additions & 6 deletions server/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -13,19 +13,17 @@ USER jboss

RUN cd /opt/jboss/ && curl -L https://downloads.jboss.org/keycloak/$KEYCLOAK_VERSION/keycloak-$KEYCLOAK_VERSION.tar.gz | tar zx && mv /opt/jboss/keycloak-$KEYCLOAK_VERSION /opt/jboss/keycloak


#Enabling Proxy address forwarding so we can correctly handle SSL termination in front ends
#such as an OpenShift Router or Apache Proxy
RUN sed -i -e 's/<http-listener /& proxy-address-forwarding="${env.PROXY_ADDRESS_FORWARDING}" /' $JBOSS_HOME/standalone/configuration/standalone.xml


ADD docker-entrypoint.sh /opt/jboss/

ADD setLogLevel.xsl /opt/jboss/keycloak/
RUN java -jar /usr/share/java/saxon.jar -s:/opt/jboss/keycloak/standalone/configuration/standalone.xml -xsl:/opt/jboss/keycloak/setLogLevel.xsl -o:/opt/jboss/keycloak/standalone/configuration/standalone.xml

ENV JBOSS_HOME /opt/jboss/keycloak

#Enabling Proxy address forwarding so we can correctly handle SSL termination in front ends
#such as an OpenShift Router or Apache Proxy
RUN sed -i -e 's/<http-listener /& proxy-address-forwarding="${env.PROXY_ADDRESS_FORWARDING}" /' $JBOSS_HOME/standalone/configuration/standalone.xml

EXPOSE 8080

ENTRYPOINT [ "/opt/jboss/docker-entrypoint.sh" ]
Expand Down
2 changes: 1 addition & 1 deletion server/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ When starting the Keycloak instance you can pass a number an environment variabl

When running Keycloak behind a proxy, you will need to enable proxy address forwarding.

docker run -e PROXY_ADDRESS_FORWARDING true jboss/keycloak
docker run -e PROXY_ADDRESS_FORWARDING=true jboss/keycloak

## Other details

Expand Down

0 comments on commit a8b1491

Please sign in to comment.