Skip to content
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

feat: unibroker add dynamic param injection on req ctx [HYB-915] #941

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

aarlaud
Copy link
Contributor

@aarlaud aarlaud commented Mar 4, 2025

What does this PR do?

Adds support for dynamic parameters injection based on specific context contained in the request.

Where should the reviewer start?

This addition gives the ability to have request specific parameter changes based on context while using an existing connection. It allows to specify a broker connection configuration once as common base for many integrations, while specific context can inject different values when necessary, based on information passed via header from the server.

How should this be manually tested?

Still WIP, need broker cloud configuration work.

Any background context you want to provide?

This mechanism allows the client to read a context ID to inject/override specific parameters from based on a context ID value passed from the server via header.

@aarlaud aarlaud force-pushed the feat/dynamic-param-inject-based-on-req-ctx branch from 12cbfb8 to becf8e6 Compare March 4, 2025 16:47
@aarlaud aarlaud changed the title feat: unibroker add dynamic param injection on req ctx feat: unibroker add dynamic param injection on req ctx [HYB-915] Mar 5, 2025
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.

1 participant