-
-
Notifications
You must be signed in to change notification settings - Fork 3
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 support to pass api key #8
Comments
@basantPayU: Did you check https://github.com/koenbuyens/securityheaders? And, did you try invoking curl from the command line? Example: curl -H "x-api-key: <API-KEY>" "https://api.securityheaders.com/?q=<URL>&hide=on&followRedirects=on" | jq -r '.summary.grade' |
The above seems to work just fine. So this issue is up for grabs :) |
I made a PR draft. This is breaking though because the order of params is changed. What do you think @iamazeem @basantPayU |
@marcuslindblom Looks good to me! |
Looks good! 👍 BTW, I just released this composite action: As of now, docker actions run only on the Linux runners. Would love to hear some feedback from you both. 😄 |
According to docs of security headers, we need to pass an API key for the request, but the current implementation doesn't provide a way to do so.
P.S. Refer to the image
The text was updated successfully, but these errors were encountered: