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

Fix security scan vulnerabilities. #928

Merged
merged 5 commits into from
Jul 19, 2023

Conversation

changxiaohui
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Jun 27, 2023

Codecov Report

Merging #928 (dfdfdf8) into master (9d2ae62) will increase coverage by 0.33%.
The diff coverage is 88.88%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #928      +/-   ##
============================================
+ Coverage     41.90%   42.23%   +0.33%     
- Complexity     1535     1549      +14     
============================================
  Files           233      234       +1     
  Lines         12878    12886       +8     
  Branches       1721     1721              
============================================
+ Hits           5396     5443      +47     
+ Misses         6802     6756      -46     
- Partials        680      687       +7     
Flag Coverage Δ
aggregator 28.11% <ø> (ø)
api 65.52% <88.88%> (+0.05%) ⬆️
infoblox 54.04% <ø> (ø)
labsdb 24.84% <ø> (ø)
management 5.67% <ø> (ø)
nlyte 30.86% <ø> (ø)
openmanage 45.17% <ø> (+3.12%) ⬆️
poweriq 42.21% <ø> (ø)
vc 41.70% <ø> (+0.27%) ⬆️
vro 5.46% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...om/vmware/flowgate/controller/AssetController.java 53.24% <ø> (ø)
...owgate/management/ManagementPortalApplication.java 25.00% <ø> (ø)
...va/com/vmware/flowgate/config/CouchbaseConfig.java 87.50% <87.50%> (ø)
...com/vmware/flowgate/config/CrossoOriginConfig.java 100.00% <100.00%> (ø)

... and 3 files with indirect coverage changes

Signed-off-by: Yongchun Chang<yongchunc@vmware.com>
Signed-off-by: Yongchun Chang<yongchunc@vmware.com>
Signed-off-by: Yongchun Chang<yongchunc@vmware.com>
Signed-off-by: Yongchun Chang<yongchunc@vmware.com>
@changxiaohui changxiaohui force-pushed the fix-security-vul branch 10 times, most recently from 3225bd3 to 1a195bb Compare July 18, 2023 09:50
@changxiaohui changxiaohui force-pushed the fix-security-vul branch 3 times, most recently from aae9d3d to 51e652b Compare July 19, 2023 06:52
Signed-off-by: Yongchun Chang<yongchunc@vmware.com>
@changxiaohui changxiaohui merged commit a1f561d into vmware:master Jul 19, 2023
20 checks passed
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

3 participants