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

Add possibility import OWASP Dependency Scans #2791

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

Hellgartner
Copy link
Contributor

@Hellgartner Hellgartner commented Feb 14, 2025

Summary of changes

Add the functionality to import OWASP Dependency scans (json) directly into opossum-UI

Context and reason for change

  • We are currently trying to simplify the workflow for OpossumUI users
  • This allows the user to open the OWASP Dependency Scan directly from OpossumUI
  • closes Implement in Opossum UI #2788

How can the changes be tested

  • CI
  • Go to Menu -> Import -> OWASP and import the file following the dialog

Note: Please review the guidelines for contributing to this repository.

Signed-off-by: Dominikus Hellgartner <[email protected]>
Signed-off-by: Dominikus Hellgartner <[email protected]>
@Hellgartner Hellgartner marked this pull request as ready for review February 14, 2025 13:21
import {
convertOwaspToOpossum,
convertScancodeToOpossum,
} from '../convertToOpossum';

describe('successfulConversionOfScanCodeFile', () => {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This testsuite needs a more general name now, as it tests both scancode and owasp. Maybe just call it 'opossum-file' or 'convertToOpossum' like the file?

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.

Implement in Opossum UI
2 participants