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.

Sign up for GitHub

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

Jump to bottom

fix(openapi): functions must not produce multiple output for same inputs #6901

Merged
merged 30 commits into from
Mar 1, 2024
Merged

fix(openapi): functions must not produce multiple output for same inputs #6901

merged 30 commits into from
Mar 1, 2024

Conversation

ArturRibeiro-CX
Copy link
Contributor

@ArturRibeiro-CX ArturRibeiro-CX commented Feb 23, 2024 •
edited
Loading

Closes #6787

Proposed Changes

  • functions must not produce multiple output for same inputs

I submit this contribution under the Apache-2.0 license.

@github-actions github-actions bot added the openapi OpenAPI query label Feb 23, 2024
@github-actions GitHub Actions
Copy link
Contributor

kics-logo

KICS version: v1.7.12

Category Results
HIGH HIGH 0
MEDIUM MEDIUM 0
LOW LOW 0
INFO INFO 0
TRACE TRACE 0
TOTAL TOTAL 0
Metric Values
Files scanned placeholder 1
Files parsed placeholder 1
Files failed to scan placeholder 0
Total executed queries placeholder 49
Queries failed to execute placeholder 0
Execution time placeholder 1

@ArturRibeiro-CX ArturRibeiro-CX self-assigned this Feb 23, 2024
ArturRibeiro-CX and others added 20 commits February 23, 2024 10:23
@ArturRibeiro-CX
Merge branch 'master' into openapi_6787
5d0e2d7
@ArturRibeiro-CX
Merge branch 'master' of https://github.com/Checkmarx/kics into opena…
41228ac 
…pi_6787
@ArturRibeiro-CX
Merge branch 'openapi_6787' of https://github.com/Checkmarx/kics into…
a041cfe 
… openapi_6787
@ArturRibeiro-CX
Merge branch 'master' into openapi_6787
e1fdd73
@ArturRibeiro-CX
Merge branch 'master' into openapi_6787
b1b4a3a
@ArturRibeiro-CX
Merge branch 'openapi_6787' of https://github.com/Checkmarx/kics into…
e1aa5a5 
… openapi_6787
@ArturRibeiro-CX
only unique keys given to names
dd69b27
@ArturRibeiro-CX
Merge branch 'master' of https://github.com/Checkmarx/kics into opena…
e2006e2 
…pi_6787
@ArturRibeiro-CX
save progress, not done yet
bcf33ac
@ArturRibeiro-CX
Merge branch 'master' of https://github.com/Checkmarx/kics into opena…
39c54fd 
…pi_6787
@ArturRibeiro-CX
changes to rego policy
3e44711
@ArturRibeiro-CX
Merge branch 'master' of https://github.com/Checkmarx/kics into opena…
9548a9d 
…pi_6787
@ArturRibeiro-CX
progress to fix query values
9402970
@ArturRibeiro-CX
Merge branch 'master' of https://github.com/Checkmarx/kics into opena…
580cae9 
…pi_6787
@ArturRibeiro-CX
merge master and changes to rego query
c94b9bb
@ArturRibeiro-CX
already gets all the values, but some of them are duplicated
c1cda48
@ArturRibeiro-CX
Merge branch 'master' of https://github.com/Checkmarx/kics into opena…
cab1f34 
…pi_6787
@ArturRibeiro-CX
changes to rego query and merge master
2c3d52d
@ArturRibeiro-CX
fix query multiple output for same input error
b447ce7
@ArturRibeiro-CX
last change to better suit the logic pre-existent
ccfe6f6
EduardoSemanas
EduardoSemanas previously approved these changes Feb 27, 2024
View reviewed changes
@asofsilva asofsilva merged commit e3dea8f into master Mar 1, 2024
22 checks passed
@asofsilva asofsilva deleted the openapi_6787 branch March 1, 2024 10:50
@BrewTestBot BrewTestBot mentioned this pull request Apr 18, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@EduardoSemanas EduardoSemanas EduardoSemanas approved these changes

Assignees

@ArturRibeiro-CX ArturRibeiro-CX

Labels
openapi OpenAPI query
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

type_has_invalid_keyword(openApi): functions must not produce multiple outputs for same inputs
3 participants
@ArturRibeiro-CX @EduardoSemanas @asofsilva