Skip to content

Conversation

@lkxiaolou
Copy link
Contributor

fastjson 1.2.58 has security problem

Does this pull request fix one issue?

None

Describe how you did it

update fastjson version to 1.2.60

Describe how to verify it

refer to https://github.com/alibaba/fastjson/releases

@codecov-io
Copy link

Codecov Report

Merging #1045 into master will increase coverage by 0.07%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1045      +/-   ##
============================================
+ Coverage     42.61%   42.68%   +0.07%     
- Complexity     1470     1474       +4     
============================================
  Files           317      317              
  Lines          9286     9286              
  Branches       1267     1267              
============================================
+ Hits           3957     3964       +7     
+ Misses         4836     4831       -5     
+ Partials        493      491       -2
Impacted Files Coverage Δ Complexity Δ
...p/sentinel/datasource/consul/ConsulDataSource.java 65.21% <0%> (-4.35%) 7% <0%> (ø)
...tinel/slots/block/flow/param/ParamFlowChecker.java 55.4% <0%> (+2.7%) 29% <0%> (+1%) ⬆️
...a/csp/sentinel/slots/statistic/base/LeapArray.java 70.29% <0%> (+2.97%) 34% <0%> (+1%) ⬆️
...m/alibaba/csp/sentinel/log/DateFileLogHandler.java 57.57% <0%> (+3.03%) 7% <0%> (+1%) ⬆️
...ava/com/alibaba/csp/sentinel/node/ClusterNode.java 100% <0%> (+4.76%) 8% <0%> (+1%) ⬆️

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 6f5ede8...f6b4be6. Read the comment docs.

Copy link
Member

@sczyh30 sczyh30 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sczyh30 sczyh30 merged commit 4d9a5ef into alibaba:master Sep 16, 2019
@sczyh30
Copy link
Member

sczyh30 commented Sep 16, 2019

Thanks for contributing!

@sczyh30 sczyh30 changed the title Update fastjson version Update fastjson version to 1.2.60 Sep 16, 2019
hughpearse pushed a commit to hughpearse/Sentinel that referenced this pull request Jun 2, 2021
CST11021 pushed a commit to CST11021/Sentinel that referenced this pull request Nov 3, 2021
[RIP-9] Fixed Design_Remoting.md issues no 103-105
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