-
Notifications
You must be signed in to change notification settings - Fork 70
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
AGNT-586 Add support for simplified key delivery #811
Conversation
symphony-bdk-core/build.gradle
Outdated
@@ -75,7 +75,7 @@ dependencies { | |||
} | |||
|
|||
// OpenAPI code generation | |||
def apiBaseUrl = "https://raw.githubusercontent.com/finos/symphony-api-spec/30fcab0fe6eaa26dcc46e7dc5909467332ec8d0d" | |||
def apiBaseUrl = "https://raw.githubusercontent.com/FabienVSymphony/symphony-api-spec/refs/heads/AGNT-585/" |
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 should be finos link
import java.util.regex.Matcher; | ||
import java.util.regex.Pattern; | ||
|
||
@Slf4j |
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.
java doc pls
|
||
import org.apiguardian.api.API; | ||
|
||
@API(status = API.Status.INTERNAL) |
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.
java doc pls
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.
by adding java docs on the two new classes, we are good.
Description
Closes #[AGNT-585]
When simplified key delivery is enabled and agent is running a version >= 24.12 we stop retrieving the keymanager token. This allows the user to run bots on cloud where an on premise key manager is not available
Dependencies
We depend on finos/symphony-api-spec#199 being merged before we can merge this PR as we use the swagger definitions to generate our APIs