-
Notifications
You must be signed in to change notification settings - Fork 15
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
Add the ability to override global Service configuration per invocation using Tool definition #185
Comments
@luigidemasi I have been thinking about this one and also #186 and #187. One thing that I've considered for #187 was to add
Then, we could bundle all these changes so that
The idea here is that the What do you think? With that, we could simplify and consolidate a lot of the handling of these parameters. |
I like it, but I would go further, I would add the "xxxxx": {
"type": "string",
"description": "configuration xxxx for the service",
"class": "configuration",
"scope": "service",
"value": "valueXXXX"
},
"zzzz": {
"type": "string",
"description": "header zzzz for the service",
"class": "header",
"scope": "service",
"value": "valueZZZZ"
},
"aaaa": {
"type": "string",
"description": "configuration aaaa for the service endpoint",
"class": "configuration",
"scope": "service-endpoint",
"value": "valueAAAA"
} wdyt? |
This is much better. +1 from me. |
I'm going to go ahead and merge #187. Once we have this fix in the code, then I can do a second pass and improve the URI building/parsing code so that I can leverage the new properties. |
…onfiguration per invocation using Tool definition
…n per invocation using Tool definition
…n per invocation using Tool definition
…ion per invocation using Tool definition
support the configuration override like this one:
The text was updated successfully, but these errors were encountered: