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.

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

TrustManagerFactory relies on default #91

Open
akwick opened this issue Apr 29, 2022 · 0 comments
Open

TrustManagerFactory relies on default #91

akwick opened this issue Apr 29, 2022 · 0 comments

Comments

@akwick
Copy link

akwick commented Apr 29, 2022

I am reaching out to you as we conducted an empirical study to understand the nature of cryptographic misuses in enterprise-driven projects on GitHub. During our study, we randomly inspected a few of the misuses. One of the misuses for which we could confirm the finding of the analysis, CogniCryptSAST is within this project.

  • HttpClientInit: The implementation in line 143 calls the method TrustManagerFactory.getDefaultAlgorithm() that relies on defaults and can be altered at runtime JCA. Thus, analyses like CogniCryptSAST consider these usages as insecure as the used TrustManager may be insecure.

We hope that this report helps you and would be glad to get your thoughts on this issue.

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

No branches or pull requests

1 participant