-
Notifications
You must be signed in to change notification settings - Fork 6
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
Cannot construct Camunda8 using explicit configuration #153
Comments
Reproducer here: https://github.com/jwulf/repro-153 |
The OAuthProvider test should contain a test that scrubs the env and hydrates only through explicit config. The smoke test needs to instantiate an OAuth component. It should also instantiate a Camunda8 object with explicit config and no env vars. It also needs to scrub the process environment vars to test explicit configuration. |
@jwulf and I paired and we think it's an issue with how the configuration object is being passed in. When I construct the object like this, I get no errors: const camunda = new Camunda8({
ZEEBE_ADDRESS: 'zeebeAddress',
ZEEBE_CLIENT_ID: 'camundaClientId',
ZEEBE_CLIENT_SECRET: 'camundaClientSecret',
CAMUNDA_OAUTH_URL: 'https://login.cloud.camunda.io/oauth/token',
}) The README.md suggests passing the configuration as a |
Give a correct example constructor of Camunda8 in the README fixes #153
Give a correct example constructor of Camunda8 in the README fixes #153
With no
CAMUNDA_OAUTH_URL
set in the env, constructing a Camunda8 object like this:Throws:
The text was updated successfully, but these errors were encountered: