pingidentity / pingidentity-cdr-sandbox

Other
5 stars 12 forks source link

unexpected character "-" in variable name near CDR_REGISTER-server-baseurl #96

Closed RyanGhd closed 2 years ago

RyanGhd commented 2 years ago

Hi team,

We are trying to prepare a demo using PING CDR sandbox that shows PING integration with our Open banking APIs. Docker throws the following error when we run the compose yaml file:

docker-compose -f docker-compose.yaml logs -f

Error

unexpected character "-" in variable name near "CDR_REGISTER-server-baseurl=\"https://mockregister.data-holder.local\"
CDR_REGISTER-pki-base-url=\"http://cdrregister:8084\"
CDR_REGISTER-ca-keystore-commonname=\"cdr-register.local\"
CDR_REGISTER-server-allowed-origins=\"https://mockregister.data-holder.local\"
SOFTWARE_STATEMENT_ACCC_JWKS_ENDPOINT=\"http://cdrregister:8084/registration/JWKS\"
SOFTWARE_STATEMENT_ACCC_ISSUER=\"cdr-register\"
SOFTWARE_STATEMENT_BANK_AUDIENCE=\"https://data-holder\"
PF_BASE_HOST=\"sso.data-holder.local\"
PF_BASE_PORT=443
DA_CLIENTID=admin_delegator
BASE_URL=http://dr.data-recipient.local:8080

CDR_REGISTER_BASE_URL=http://cdrregister:8084
DR_CLIENT-authorization_endpoint=https://sso.data-holder.local/as/authorization.oauth2
DR_CLIENT-ss_redirect_uri=http://dr.data-recipient.local:8080/*
CONSENT_APP_server_baseurl=https://consent.data-holder.local
CONSENT_APP_account_retriever_class_impl=com.pingidentity.ps.cdr.consentapp.impl.mock.account.MockAPIAccountRetriever
CONSENT_APP_account_retriever_configuration={ \"baseUrl\": \"http://mock-dh-api:6879\", \"ignore-ssl\": true }
CONSENT_APP_external_consent_class_impl=com.pingidentity.ps.cdr.consentapp.impl.standard.extconsent.NoExternalConsentStorageImpl
CONSENT_APP_external_consent_configuration={}
CONSENT_APP_pf_baseurl=https://sso.data-holder.local
CONSENT_APP_pf_baseurl_backend=https://pingfederate:9031
CONSENT_APP_pf_oauth_client_mgt_username=Administrator
CONSENT_APP_pf_oauth_client_mgt_password=2FederateM0re
CONSENT_APP_pf_agentless_username=consentappuser
CONSENT_APP_pf_agentless_password=2FederateM0re
CONSENT_BASEURL=https://consent.data-holder.local
PINGDIRECTORY_HOSTNAME=pingdirectory:1443
PINGFEDERATE_HOSTNAME=pingfederate:9031
REGISTER_STATUS_BASEURL=http://cdrregister:8084/cdr-register/v2/banking
REGISTER_JWKS_ENDPOINT=http://cdrregister:8084/.well-known/openid-configuration/JWKS
DATAHOLDER_BRAND_ID=brand-cdr-sandbox-data-in-sample-dr-0005
DATAHOLDER_SOFTWARE_ID=software-cdr-sandbox-data-in-sample-dr-0005
ADH_ID=https://sso.data-holder.local:6443
ADH2_ID=https://sso-2.data-holder.local:6443
ADR_SSO_ENDPOINT=sso.data-holder.local:6443
ADR_API_ENDPOINT=api.data-holder.local:6443
ADR_REDIRECT_URI=https://spa.data-recipient.local/index.html
ADR_DATAIN_LISTEN_PORT=2280
SCIM_BASEURL=https://pingdirectory:1443/scim/v2/
AT_ENCRYPTION_KEY=614E645267556B58703273357638792F423F4528472B4B6250655368566D5971
#Valid values: none, embedded, external
PDG_MODE=embedded"

Docker Desktop version: 4.5.1 Docker Compose: 2.2.3 OS: Windows 11 WSL 2 Distro: Ubuntu-20.4

ghost commented 2 years ago

Hello @RyanGhd

This could be an issue with the Docker version you are using. Could you please try disabling "Use Docker Compose V2" in Settings->General in your Docker Desktop.

I am also attaching my env file.
cdr.env.txt

Hope this helps.

RyanGhd commented 2 years ago

thanks @jasAtPing, using docker compose v1 fixed the issue. Do you have any plans to update to v2?

ghost commented 2 years ago

thats great @RyanGhd. Update to V2 is not an immediate priority but we plan to tackle it later this year.