Skip to content

[ResponseOps][Connector V2] Make getAxiosInstance function available from the action client#244619

Merged
adcoelho merged 19 commits intoelastic:mainfrom
adcoelho:expose-get-axios
Dec 5, 2025
Merged

[ResponseOps][Connector V2] Make getAxiosInstance function available from the action client#244619
adcoelho merged 19 commits intoelastic:mainfrom
adcoelho:expose-get-axios

Conversation

@adcoelho
Copy link
Copy Markdown
Contributor

@adcoelho adcoelho commented Nov 28, 2025

Closes #243808

Summary

Makes the utility function getAxiosInstanceWithAuth accessible outside the plugin via the actionsClient.

For a given connector ID, this function validates the secrets and returns an axios instance properly configured with authentication.

@adcoelho adcoelho self-assigned this Nov 28, 2025
@adcoelho adcoelho changed the title [ResponseOps][Connector V2] Make getAxiosInstance function available from the action type registry [ResponseOps][Connector V2] Make getAxiosInstance function available from the action client Nov 28, 2025
@adcoelho adcoelho marked this pull request as ready for review December 1, 2025 13:07
@adcoelho adcoelho requested a review from a team as a code owner December 1, 2025 13:07
Copy link
Copy Markdown
Contributor

@ymao1 ymao1 left a comment

Choose a reason for hiding this comment

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

LGTM. Would be good to see a functional test that tests getting a preconfigured axios instance using the actions client.

});

// check to see if it's in memory connector before fetching secrets
const inMemoryAction = inMemoryConnectors.find(
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: is there a way to set the flag in the above check for inmemory connector?

@cnasikas cnasikas added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// labels Dec 2, 2025
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@cnasikas cnasikas added backport:version Backport to applied version labels v9.3.0 and removed backport:skip This PR does not require backporting labels Dec 2, 2025
@adcoelho adcoelho merged commit 83ceae3 into elastic:main Dec 5, 2025
12 checks passed
@kibanamachine kibanamachine added backport:skip This PR does not require backporting and removed backport:version Backport to applied version labels labels Dec 5, 2025
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

✅ unchanged

History

cc @adcoelho

adcoelho added a commit that referenced this pull request Dec 5, 2025
…ent.getAxiosInstance` (#245374)

## Summary

Updates #244619 with the changes in #245132 .
wildemat pushed a commit to wildemat/kibana that referenced this pull request Dec 5, 2025
…e from the action client (elastic#244619)

Closes elastic#243808

## Summary

Makes the utility function `getAxiosInstanceWithAuth` accessible outside
the plugin via the `actionsClient`.

For a given connector ID, this function validates the secrets and
returns an axios instance properly configured with authentication.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
wildemat pushed a commit to wildemat/kibana that referenced this pull request Dec 5, 2025
JordanSh pushed a commit to JordanSh/kibana that referenced this pull request Dec 9, 2025
…e from the action client (elastic#244619)

Closes elastic#243808

## Summary

Makes the utility function `getAxiosInstanceWithAuth` accessible outside
the plugin via the `actionsClient`.

For a given connector ID, this function validates the secrets and
returns an axios instance properly configured with authentication.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
JordanSh pushed a commit to JordanSh/kibana that referenced this pull request Dec 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ResponseOps][Worfklows][Phase 2] Make getAxiosInstance function available from the action client

5 participants