dependency(jetty): upgrade for security by halibobo1205 · Pull Request #5571 · tronprotocol/java-tron
Navigation Menu
{{ message }}
tronprotocol / java-tron Public
- Notifications You must be signed in to change notification settings
- Fork 1.6k
Merged
halibobo1205 merged 1 commit intotronprotocol:developfrom
Nov 24, 2023Merged
dependency(jetty): upgrade for security#5571
halibobo1205 merged 1 commit intotronprotocol:developfrom
dependency(jetty): upgrade for security#5571
halibobo1205 merged 1 commit intotronprotocol:developfrom
Conversation
halibobo1205
changed the title
dependency(jetty): upgrade for security.
dependency(jetty): upgrade for security
halibobo1205
added this to the
GreatVoyage-v4.7.4 milestone
halibobo1205
self-assigned this
lvs007
requested review from
317787106 and
jwrct
317787106 reviewed Nov 10, 2023
| compile 'org.eclipse.jetty:jetty-servlet:9.4.49.v20220914' | ||
| compile 'org.eclipse.jetty:jetty-server:9.4.53.v20231009' | ||
| compile 'org.eclipse.jetty:jetty-servlet:9.4.53.v20231009' | ||
| compile 'com.alibaba:fastjson:1.2.83' |
Copy link
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not solve other dependency security also but only jetty? such as bcprov-jdk15on in https://github.com/tronprotocol/libp2p/pull/74/files.
Copy link
Contributor Author
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if bcprov-jdk15on is required, it will be a stand-alone PR.
halibobo1205
force-pushed
the
dependencies/jetty
branch
from
6a91e60 to
0edc0a8
Compare
halibobo1205
requested review from
forfreeday and
tomatoishealthy
tomatoishealthy approved these changes Nov 24, 2023
halibobo1205
merged commit
be07f5c
into
tronprotocol:develop
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants