Skip to content

Conversation

@kokosing
Copy link
Contributor

Exclude org.codehaus.jackson

This libraries are old (2013) and have plenty of CVEs. They were
migrated to org.fasterxml.jackson.

@kokosing
Copy link
Contributor Author

@mneethiraj why shim modules requires so many dependencies? I see that in hive plugin as well as in hbase. The more dependencies we put the higher risk for CVEs. How can I check what is actually used? I removed some of them and I see that project (module) compiles just fine, is this good enough for testing?

@kokosing
Copy link
Contributor Author

@mneethiraj
Copy link
Contributor

why shim modules requires so many dependencies

@kokosing - you are right, shim modules only have dependency on libraries that include classes referenced in authorization interface (like HiveAuthorizer, HiveAuthorizerFactory), and ranger-plugin-classloader library.

About this patch, given org.codehaus.jackson libraries are not included in Hive plugin packaging, is it necessary to exclude them from pom.xml file?

@kokosing
Copy link
Contributor Author

kokosing commented Jul 24, 2024

About this patch, given org.codehaus.jackson libraries are not included in Hive plugin packaging, is it necessary to exclude them from pom.xml file?

I used mvn dependency:tree and I excluded it until the moment I no longer saw this dependency. So I believe it is necessary this way.

shim modules only have dependency on libraries that include classes referenced in authorization interface (like HiveAuthorizer, HiveAuthorizerFactory), and ranger-plugin-classloader library.

Is it safe to remove these dependencies and assume that if project compiles we are good? If that would be the case then we could remove plenty of dependencies.

@kumaab
Copy link
Contributor

kumaab commented Jul 25, 2024

Please rebase the PR to run all checks, 1 check is missing.

This libraries are old (2013) and have plenty of CVEs. They were
migrated to org.fasterxml.jackson.
@kokosing kokosing force-pushed the origin/master/005 branch from 80101e5 to 1bd10cb Compare July 25, 2024 11:39
@kokosing
Copy link
Contributor Author

Done

mneethiraj pushed a commit that referenced this pull request Jul 26, 2024
… library - PR #351

Signed-off-by: Madhan Neethiraj <madhan@apache.org>
mneethiraj pushed a commit that referenced this pull request Jul 26, 2024
… library - PR #351

Signed-off-by: Madhan Neethiraj <madhan@apache.org>
(cherry picked from commit e5053e1)
@mneethiraj
Copy link
Contributor

@kokosing - the patch is merged in master and ranger-2.5 branches. Thank you!

@mneethiraj mneethiraj closed this Jul 26, 2024
@kokosing kokosing deleted the origin/master/005 branch July 26, 2024 09:57
@kokosing
Copy link
Contributor Author

Thank you!

mapr-devops pushed a commit to mapr/ranger that referenced this pull request May 8, 2025
… library - PR apache#351

Signed-off-by: Madhan Neethiraj <madhan@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants