-
Notifications
You must be signed in to change notification settings - Fork 189
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
added scope to properties #4473
Conversation
👋 🤖 ✅ Looks like the changes were ported across versions, nice job! 🎉 You can read more about the versioning within our docs in our documentation guidelines. |
Checking in with the team to see if this will also be available in 8.5 as mentioned via camunda/camunda#23072 |
Duplicate of #4393, however they haven't backported that PR. |
@akeller well, one of them has to be merged. A customer is missing it. btw this is required for ms entra integration. |
I just filed a PR so that the old place where the scope was added points to the new one: camunda/camunda#23673 Btw the old property is |
@@ -103,6 +103,7 @@ Example of configuring the connection to a Self-Managed Zeebe cluster: | |||
```properties | |||
zeebe.client.cloud.clientId=your-client-id | |||
zeebe.client.cloud.clientSecret=your-client-secret | |||
zeebe.client.cloud.scope=your-scope # optional |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not released in 8.5 see the original issue camunda/camunda#23071 . It will be included in the next 8.5 patch.
We need to revert this.
Please always refer to the actual originating issue for context.
Thanks @akeller for the ping.
Description
This property can be configured, but is undocumented:
https://github.com/camunda/camunda/blob/main/clients/spring-boot-starter-camunda-sdk/src/main/java/io/camunda/zeebe/spring/client/properties/common/ApiProperties.java#L24
When should this change go live?
hold
label or convert to draft PR)PR Checklist
/versioned_docs
directory./docs
directory (aka/next/
).