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

update x-stream to 1.4.17 (from 1.4.16) #667

Closed
wants to merge 2 commits into from

Conversation

sseide
Copy link
Contributor

@sseide sseide commented Jun 7, 2021

Description

security update for com.thoughtworks.xstream:xstream from 1.4.16 to 1.4.17

Motivation and Context

This update fixes the following CVE:

How Has This Been Tested?

run gradle check and used it ourself for some days without problems

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the code style of this project.
  • I have updated the documentation accordingly.

@codecov-commenter
Copy link

codecov-commenter commented Jun 7, 2021

Codecov Report

Merging #667 (21b6b68) into master (ec1d462) will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #667      +/-   ##
============================================
+ Coverage     55.40%   55.42%   +0.01%     
- Complexity    10216    10219       +3     
============================================
  Files          1047     1047              
  Lines         64462    64462              
  Branches       7311     7311              
============================================
+ Hits          35718    35726       +8     
+ Misses        26243    26232      -11     
- Partials       2501     2504       +3     
Impacted Files Coverage Δ
...ter/protocol/http/proxy/DefaultSamplerCreator.java 54.63% <0.00%> (+0.51%) ⬆️
...n/java/org/apache/jmeter/reporters/Summariser.java 86.25% <0.00%> (+0.76%) ⬆️
...ter/protocol/http/proxy/SamplerCreatorFactory.java 41.86% <0.00%> (+13.95%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ec1d462...21b6b68. Read the comment docs.

@FSchumacher
Copy link
Contributor

Thanks for the PR. It has been merged into master.

@FSchumacher FSchumacher closed this Jun 7, 2021
@sseide sseide deleted the update_xstream branch October 21, 2021 10:22
This pull request was closed.
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.

3 participants