Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update for sso provider "oauth2" in https://github.com/OpenLiberty/ci.docker/blob/master/SECURITY.md #159

Closed
meiaus opened this issue May 17, 2020 · 1 comment

Comments

@meiaus
Copy link

meiaus commented May 17, 2020

Not sure if the following SECURITY.md page is handled via "open-liberyt-operator" repo, but since the following docker build problem is experienced from running with Open Liberty Operator, tracking this ticket here:

https://github.com/OpenLiberty/ci.docker/blob/master/SECURITY.md

Recently when running docker build with Dockerfile which was tested in April, e.g.

FROM openliberty/open-liberty:full-java8-openj9-ubi

# Optional functionality
ARG TLS=true
ARG SEC_SSO_PROVIDERS="oauth oidc facebook google github twitter linkedin"
#ARG OPENJ9_SCC=false
ARG VERBOSE=true

COPY --chown=1001:0  server.xml /config/
COPY --chown=1001:0  AcmeWebClientEar.ear /config/apps

# This script will add the requested XML snippets and gow image to be fit-for-purpose
RUN configure.sh

the following failure appeared:

++ grep oauth
+ [[ -n sso-oauth2.xml ]]
+ cp /opt/ol/helpers/build/configuration_snippets/sso-oauth.xml /config/configDropins/defaults
cp: cannot stat '/opt/ol/helpers/build/configuration_snippets/sso-oauth.xml': No such file or directory

After consulting with @brutif and @leochr, it's realized, the sso provider "oauth" has been changed to "oauth2".
After updating Dockerfile to reflect the new provider "oauth2",

ARG SEC_SSO_PROVIDERS="oauth2 oidc facebook google github twitter linkedin"

the problem is the resolved.

Please update the readme accoringly:
https://github.com/OpenLiberty/ci.docker/blob/master/SECURITY.md

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants