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 support for more GIT SCMs in finding view for the finding URL #9710

Merged
merged 4 commits into from
Apr 30, 2024

Conversation

eu-david
Copy link
Contributor

Description

Reworked the finding URL generation method to cover more GIT SCMs (github, gitlab, gitea, codeberg, bitbucket) by default. Also unified the bitbucket and github link generation methods into 1.

Test results

Added two new tests to test the new functionality, make sure that the function generates the correct URL for gitea.

Copy link

dryrunsecurity bot commented Mar 11, 2024

Hi there 👋, @DryRunSecurity here, below is a summary of our analysis and findings.

DryRun Security Status Findings
Sensitive Functions Analyzer 0 findings
Configured Sensitive Files Analyzer 3 findings
Sensitive Files Analyzer 1 findings

Note

🔴 Risk threshold exceeded. Adding a reviewer if one is configured in .dryrunsecurity.yaml.

notification list: @mtesauro @grendel513

Tip

Get answers to your security questions. Add a comment in this PR starting with @DryRunSecurity. For example...

@dryrunsecurity What are common security issues with web application cookies?

Powered by DryRun Security

@github-actions github-actions bot added the docs label Mar 14, 2024
Copy link
Contributor

@mtesauro mtesauro left a comment

Choose a reason for hiding this comment

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

Approved

Copy link
Contributor

@hblankenship hblankenship left a comment

Choose a reason for hiding this comment

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

Now THAT is a function name...
test_get_file_path_with_link_and_source_code_management_uri_gitea_or_codeberg_project_with_commithash_and_line

@mtesauro mtesauro merged commit 71811c7 into DefectDojo:dev Apr 30, 2024
120 of 121 checks passed
dogboat pushed a commit to dogboat/django-DefectDojo that referenced this pull request May 6, 2024
…fectDojo#9710)

* Add support for more GIT SCMs

* fix flake8 issue

* docs: added other supported SCM to the documentation

---------

Co-authored-by: Eugeniu David <eugeniu.david@mic-cust.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants