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

Add display parameter to getScanResults (API) #479

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DanielKrasny
Copy link

The actual output of /api/v1/getScanResults is an object, where value contains next object with test result properties. These properties contain a name key, which value is identical to name of superior's object key.

Example (simplified for clarity):

{
    "content-security-policy": {
        "name": "content-security-policy"
    }
}

For my use case, I found an alternative keeping only the value objects and passing them into array. The name remained only in the properties object. Of course, I don't want to break current code, so this option is available as an optional parameter.

Example (simplified for clarity):

[
    {
        "name": "content-security-policy"
    }
]

The initiative comes from no availability to iterate through object keys in programming language that I currently use.

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.

None yet

1 participant