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.

Sign up for GitHub

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

Jump to bottom

Please consider using a minimal set of netty dependencies instead of netty-all. #242

Open
dokky opened this issue Oct 7, 2020 · 1 comment
Open

Please consider using a minimal set of netty dependencies instead of netty-all. #242

dokky opened this issue Oct 7, 2020 · 1 comment
Assignees
@dbl-x

Comments

@dokky
Copy link

dokky commented Oct 7, 2020

Describe the bug
Please consider using a minimal set of netty dependencies instead of netty-all.

That pulls all available submodules from netty polluting classpath and complicates dependency management.
Plus current version of netty has security vulnerabilities, you can scan with OWASP dependency management maven plugin to get a full report.

Please let me know if you want a PR with the fix.
@dbl-x
Copy link
Contributor

dbl-x commented Aug 23, 2021

Very good suggestion, you are welcome to submit a PR to simplify the dependence on Netty.

@dbl-x dbl-x self-assigned this Aug 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dbl-x dbl-x

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dokky @dbl-x