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

Pr ts feature/shard manager #414

Merged
merged 16 commits into from
Sep 10, 2020
Merged

Pr ts feature/shard manager #414

merged 16 commits into from
Sep 10, 2020

Conversation

tsunez
Copy link
Contributor

@tsunez tsunez commented Sep 9, 2020

Added shard manager support to CxFlow. The change make it so determineTeamAndOwnerID notifies Shard Manager of the current team and project name when new scans arrive.

AOP had incorrect function pattern

tsunez and others added 10 commits September 1, 2020 14:40
Added shard manager support to CxFlow. The change make it so determineTeamAndOwnerID notifies Shard Manager of the current team and project name when new scans arrive.
Added shard manager support to CxFlow. The change make it so determineTeamAndOwnerID notifies Shard Manager of the current team and project name when new scans arrive.
…kmarx-ltd/cx-flow into pr-ts-feature/shard-manager

# Conflicts:
#	build.gradle
Added shard manager support to CxFlow. The change make it so determineTeamAndOwnerID notifies Shard Manager of the current team and project name when new scans arrive.
@tsunez tsunez requested a review from AvivCx September 9, 2020 20:48
@tsunez tsunez closed this Sep 9, 2020
@tsunez tsunez reopened this Sep 9, 2020
Test was not handling shard manager variable correctly.
Test wasn't passing required ShardManagerTracker into ScanRequestconverter.
Test configuration was accidentally committed.
@sonarcloud
Copy link

sonarcloud bot commented Sep 10, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

warning The version of Java (1.8.0_265) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11.
Read more here

@tsunez tsunez closed this Sep 10, 2020
@AvivCx AvivCx reopened this Sep 10, 2020
@AvivCx AvivCx added the enhancement New feature or request label Sep 10, 2020
@AvivCx AvivCx merged commit 635a706 into develop Sep 10, 2020
AvivCx pushed a commit that referenced this pull request Sep 22, 2020
…ances (#414)

Add shard-manager in CxFlow SAST client to support multiple SAST instances
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants