Skip to content

fix: Use https:// URLs in report templates#5582

Merged
jeremylong merged 1 commit intodependency-check:mainfrom
Marcono1234:report-templates-https-urls
Mar 23, 2023
Merged

fix: Use https:// URLs in report templates#5582
jeremylong merged 1 commit intodependency-check:mainfrom
Marcono1234:report-templates-https-urls

Conversation

@Marcono1234
Copy link
Copy Markdown
Contributor

@Marcono1234 Marcono1234 commented Mar 22, 2023

Fixes Issue

none

Description of Change

  • Update URLs in report templates to use https:// instead of http://
  • Update HTML and Jenkins report template to also convert https:// license URLs to links

Have test cases been added to cover the new functionality?

no

@boring-cyborg boring-cyborg Bot added the core changes to core label Mar 22, 2023
@Marcono1234 Marcono1234 changed the title Use https:// URLs in report templates fix: Use https:// URLs in report templates Mar 22, 2023
@jeremylong jeremylong added this to the 8.2.1 milestone Mar 23, 2023
@jeremylong jeremylong merged commit 32aa38f into dependency-check:main Mar 23, 2023
@jeremylong
Copy link
Copy Markdown
Collaborator

Thanks for the PR!

@Marcono1234 Marcono1234 deleted the report-templates-https-urls branch March 23, 2023 19:35
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Dec 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

core changes to core

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants