Skip to content

feature: custom endpoint host for AWS Embedding Node#6264

Open
rohan-patil2 wants to merge 2 commits intoFlowiseAI:mainfrom
rohan-patil2:feature/custom-embedding-host-bedrock
Open

feature: custom endpoint host for AWS Embedding Node#6264
rohan-patil2 wants to merge 2 commits intoFlowiseAI:mainfrom
rohan-patil2:feature/custom-embedding-host-bedrock

Conversation

@rohan-patil2
Copy link
Copy Markdown
Contributor

The custom endpoint host for the AWS Embedding node was missing, which prevented connections to the cross-account AWS Bedrock environment.

Screenshot 2026-04-21 at 9 23 33 PM

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the AWS Bedrock Embedding node to version 5.1 and introduces a 'Custom Endpoint Host' configuration option, allowing users to override the default AWS Bedrock endpoint. It also refactors the model selection logic to use an 'effectiveModel' variable for consistency. A review comment correctly identified a potential runtime error where 'effectiveModel' could be undefined if no model is provided, which would cause a crash when checking for Cohere models; a suggestion was provided to throw an explicit error in this case.

Comment thread packages/components/nodes/embeddings/AWSBedrockEmbedding/AWSBedrockEmbedding.ts Outdated
@harshit-flowise harshit-flowise self-requested a review April 22, 2026 16:17
@rohan-patil2 rohan-patil2 force-pushed the feature/custom-embedding-host-bedrock branch from 54dcc0a to 121b8dc Compare April 28, 2026 01:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants