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

Wrong supported engine versions #2

Open
fcoene opened this issue May 16, 2019 · 5 comments
Open

Wrong supported engine versions #2

fcoene opened this issue May 16, 2019 · 5 comments

Comments

@fcoene
Copy link

fcoene commented May 16, 2019

The [2.2, 4.3] (in your plugin.xml) do not exist in OWASP Dependency Check. The latest versions are 3.3.4, 4.0.2 or 5.0.0-M3 (https://github.com/jeremylong/DependencyCheck/releases).
Which version is the plugin written for? I can't get it to parse 5.0.0-M3 or 4.0.2 scan.json files.

@skamani
Copy link
Owner

skamani commented May 19, 2019 via email

@fcoene
Copy link
Author

fcoene commented May 20, 2019

FYI, after testing it on different environments, it seems the plugin does work with the latest scan versions but it fails on SSC servers with an Oracle backend due to more strict parameter types & automatic conversions.

@skamani
Copy link
Owner

skamani commented May 20, 2019 via email

@fcoene
Copy link
Author

fcoene commented May 20, 2019

The scan processing fails on a "org.hibernate.engine.jdbc.spi.SqlExceptionHelper" error due to an Oracle ORA-06553 PLS-306 "wrong number or types of arguments in call to UPDATEEXISTINGWITHLATEST" exception.

@fcoene
Copy link
Author

fcoene commented May 23, 2019

I solved it. It turns out that your plugin doesn't really parse the scanDate. The scanDate case label is never hit as it is embedded under the projectInfo json block, so I think it stays uninitialized. It seems that mysql has no problem saving this but Oracle does.
When you look at the engineVersion, it is embedded in the scaninfo block, so you have a function to extract it. scanDate is also embedded in a json block (projectInfo), so you need a function to extract this as well.

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

2 participants