-
Notifications
You must be signed in to change notification settings - Fork 550
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
build: upgrade gjson to v1.6.7 #1366
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1366 +/- ##
==========================================
+ Coverage 62.62% 62.83% +0.21%
==========================================
Files 41 41
Lines 2360 2360
==========================================
+ Hits 1478 1483 +5
+ Misses 706 703 -3
+ Partials 176 174 -2
Continue to review full report at Codecov.
|
cc @starsz to review |
and the front e2e failed: https://github.com/apache/apisix-dashboard/pull/1366/checks?check_run_id=1756969309#step:8:289 need @juzhiyuan confirm |
has been merged, @ThanoshanMV you can merge the master branch, fixed the e2e test cases of FE |
@ThanoshanMV Please sync with the latest codes from the master branch. |
Okay, I'll do it @juzhiyuan |
I just did it for you ^_^ |
Please answer these questions before submitting a pull request
Why submit this pull request?
Bugfix
New feature provided
Improve performance
Backport patches
Related issues
fixes There is a vulnerability in gjson v1.6.1,upgrade recommended #1353
How to fix?
Upgraded version gjson to
v1.6.7