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

tests: emulate execution of pdfbox-tools with cyclonedx-maven-plugin and depscan #69

Merged
merged 3 commits into from
Aug 28, 2023

Conversation

algomaster99
Copy link
Member

pdfbox-app is an executable from https://github.com/apache/pdfbox. We try to execute its composite jar that is found in pdfbox-tools (as hinted from here).

Then I run the main class and add dependencies after seeing the error log and ensuring that the dependency is listed somewhere in pom, parent-pom, or poms of direct dependencies.

I used mvn org.cyclonedx:cyclonedx-maven-plugin:makePackageBom inside pdfbox/tools to create a BOM for only that module. However, cyclonedx maven plugin does not fetch any dependency for it even though there exists some.

@algomaster99
Copy link
Member Author

algomaster99 commented Aug 28, 2023

Since the root component is not reported by depscan, our tool outputs that it is not present:

[NOT WHITELISTED]: org/apache/pdfbox/tools/PDFBox

depscan, although, mentions that this dependency exists under dependencies. See https://github.com/ASSERT-KTH/terminator/pull/69/files#diff-72de8ed0483e65162662aa3dbd7d49c4aed024e849ef1134fbf1fcc6161fa89aR592.

@algomaster99 algomaster99 changed the title tests: emulate execution of pdfbox-tools with cyclonedx-maven plugin sbom tests: emulate execution of pdfbox-tools with cyclonedx-maven-plugin and depscan Aug 28, 2023
@algomaster99
Copy link
Member Author

@MartinWitt have a look it. Interesting case.

@algomaster99 algomaster99 merged commit a4f7116 into main Aug 28, 2023
6 checks passed
@algomaster99 algomaster99 deleted the level2 branch August 28, 2023 16:35
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

Successfully merging this pull request may close these issues.

1 participant