-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Security solution] Improve AI connector error handling #167674
Conversation
Pinging @elastic/security-threat-hunting (Team:Threat Hunting) |
Pinging @elastic/security-solution (Team: SecuritySolution) |
@@ -21,7 +21,7 @@ export function getConnectorType(): BedrockConnector { | |||
id: BEDROCK_CONNECTOR_ID, | |||
iconClass: lazy(() => import('./logo')), | |||
selectMessage: i18n.translate('xpack.stackConnectors.components.bedrock.selectMessageText', { | |||
defaultMessage: 'Send a request to AWS Bedrock systems.', | |||
defaultMessage: 'Send a request to AWS Bedrock.', |
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.
The commit got lost for these couple of translations from the renaming PR, so I am tucking them in here. hope that is ok
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.
response ops changes lgtm
@elasticmachine merge upstream |
💛 Build succeeded, but was flaky
Failed CI Steps
Test Failures
Metrics [docs]Module Count
Public APIs missing comments
Async chunks
Page load bundle
History
To update your PR or re-run it, just comment with: |
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.
LGTM
Improves error handling on Generative AI connectors.
To test:
*.ndjson