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

Paging support for azure wrapper #575

Merged
merged 15 commits into from
Sep 4, 2023

Conversation

igorlombacx
Copy link
Contributor

@igorlombacx igorlombacx commented Aug 29, 2023

By submitting a PR to this repository, you agree to the terms within the Checkmarx Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

Description

Paging support for azure requests.

before paging:
image

after paging:
image

Command example:

./cx utils contributor-count azure --url-azure https://url.test --orgs Checkmarx --token T0K3N51N15tR0CH3CKM4RX --debug

References

https://checkmarx.atlassian.net/browse/AST-28003?atlOrigin=eyJpIjoiZmYwMDAyMDVmZTkyNDg2MTg1MTg4MTY1MGU4ZDZiYjIiLCJwIjoiaiJ9

Testing

integration tests

Checklist

  • I have added documentation for new/changed functionality in this PR (if applicable).
  • I have updated the CLI help for new/changed functionality in this PR (if applicable).
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

@pedrompflopes pedrompflopes requested review from a team, margaritalm and diogopcx and removed request for a team August 29, 2023 19:25
@github-actions
Copy link

github-actions bot commented Aug 29, 2023

Logo
Checkmarx One – Scan Summary & Detailsc265bab8-1543-4d7d-9756-d35bb0bed7a0

New Issues

Severity Issue Source File / Package Checkmarx Insight
MEDIUM CVE-2023-29407 Go-golang.org/x/image-v0.0.0-20190802002840-cff245a6509b Vulnerable Package
MEDIUM CVE-2023-29408 Go-golang.org/x/image-v0.0.0-20190802002840-cff245a6509b Vulnerable Package

@igorlombacx igorlombacx marked this pull request as ready for review August 31, 2023 15:20
@pedrompflopes pedrompflopes requested a review from a team August 31, 2023 15:20
diogopcx)
- GetProjects fixed using X-Ms-Continuationtoken header
- refactor in paginateGetter function
diogopcx
diogopcx previously approved these changes Sep 4, 2023
@igorlombacx igorlombacx merged commit 66b8fda into main Sep 4, 2023
6 checks passed
@igorlombacx igorlombacx deleted the ENHANCEMENT/AST-28003-AZURE-PAGING branch September 4, 2023 17:11
igorlombacx added a commit that referenced this pull request Oct 20, 2023
* Paging support for azure wrapper

* linter

* refactoring paginateGetter

* linter and refactor

* integration test improvements

* review suggestions and fixes (thx
diogopcx)
- GetProjects fixed using X-Ms-Continuationtoken header
- refactor in paginateGetter function

* linter

* unused const removed

* GetRepositories refactor
ittaigilat-cx pushed a commit that referenced this pull request Jan 4, 2024
* Paging support for azure wrapper

* linter

* refactoring paginateGetter

* linter and refactor

* integration test improvements

* review suggestions and fixes (thx
diogopcx)
- GetProjects fixed using X-Ms-Continuationtoken header
- refactor in paginateGetter function

* linter

* unused const removed

* GetRepositories refactor
thtri pushed a commit to thtri/ast-cli that referenced this pull request Apr 19, 2024
* Paging support for azure wrapper

* linter

* refactoring paginateGetter

* linter and refactor

* integration test improvements

* review suggestions and fixes (thx
diogopcx)
- GetProjects fixed using X-Ms-Continuationtoken header
- refactor in paginateGetter function

* linter

* unused const removed

* GetRepositories refactor
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.

2 participants