Skip to content

Conversation

@pquentin
Copy link
Member

@pquentin pquentin commented Nov 24, 2025

While those APIs are private, they are needed in Elasticsearch's rest-api-spec, which we want to generate from the specification.

Note that those APIs are surprisingly similar to the Fleet secret APIs that I've added in #5729.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 24, 2025

Following you can find the validation changes against the target branch for the APIs.

API Status Request Response
connector.secret_delete 🟠 → 🟢 Missing type → 4/4 Missing type → 4/4
connector.secret_get 🟠 → 🟢 Missing type → 11/11 Missing type → 11/11
connector.secret_post 🟠 → 🟢 Missing type → 9/9 Missing type → 9/9
connector.secret_put 🟠 → 🟢 Missing type → 4/4 Missing type → 4/4

You can validate these APIs yourself by using the make validate target.

Comment on lines +21 to +23
body: {
deleted: boolean
}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines +28 to +39
export interface Request extends RequestBase {
urls: [
{
path: '/_connector/_secret/{id}'
methods: ['DELETE']
}
]
path_parts: {
/** The ID of the secret */
id: string
}
}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines +28 to +39
export interface Request extends RequestBase {
urls: [
{
path: '/_connector/_secret/{id}'
methods: ['GET']
}
]
path_parts: {
/** The ID of the secret */
id: string
}
}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines +21 to +24
body: {
id: string
value: string
}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines 22 to 38
/**
* Creates a secret for a Connector.
*
* @rest_spec_name connector.secret_post
* @availability stack stability=experimental visibility=private
*/
export interface Request extends RequestBase {
urls: [
{
path: '/_connector/_secret'
methods: ['POST']
}
]
body: {
value: string
}
}
Copy link
Member Author

Comment on lines +21 to +23
body: {
id: string
}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines +29 to +38
urls: [
{
path: '/_connector/_secret/{id}'
methods: ['PUT']
}
]
path_parts: {
/** The ID of the secret */
id: string
}
Copy link
Member Author

Comment on lines +23 to +25
body: {
result: Result
}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines +29 to +34
urls: [
{
path: '/_connector/_secret'
methods: ['POST']
}
]
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines +35 to +37
body: {
value?: string
}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

@MattDevy MattDevy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Thanks also for the comments to the elasticsearch repo for each Request / Response, it makes it really clear and easy to validate! 🎉

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants