-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Support API Key authentication for Metrics API scaler #1083
Comments
Hi @tomkerkhove, I would like to pick this issue. Based on this thread, I believe no one is doing any development on this. Just wanted to check if I can start. |
Nope, that is correct. Thanks for picking this up! |
Perfect. So what I get from the description is that we wanted to integrate API Key (which will be available in the metadata) to be appended to the http URL like this |
Today, they have to provide the full URL to call - https://keda.sh/docs/2.0/scalers/metrics-api/ In theory they can add the API key in there already but I'd prefer not to. What we want to achieve here is:
For both I would provide a standard header/query parameter name, but with the option to define their own in case they deviate from the mainstream. Make sense? |
Got it. All clear now. I am starting with this. Thanks! |
@tomkerkhove this is done. can we close this issue :) |
Thanks! |
Use-Case
Support API Key authentication for Metrics API scaler
Specification
Reference
#929
The text was updated successfully, but these errors were encountered: