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

Make OSS Index URL configurable via CLI #5176

Closed
wants to merge 1 commit into from
Closed

Conversation

matzi
Copy link

@matzi matzi commented Dec 22, 2022

Fixes Issue

Makes the OSS Index URL configurable in the CLI via --ossIndexUrl (#5036)

Description of Change

URL can be defined via --ossIndexUrl otherwise the default is used.

Have test cases been added to cover the new functionality?

No

@boring-cyborg boring-cyborg bot added the cli changes to the cli label Dec 22, 2022
@matzi matzi changed the title Make OSS Index URL configurable via CLI #5036 Make OSS Index URL configurable via CLI Dec 22, 2022
@aikebah
Copy link
Collaborator

aikebah commented Dec 23, 2022

Thanks for the PR. CLI options also need to be maunually documented within https://github.com/jeremylong/DependencyCheck/blob/main/cli/src/site/markdown/arguments.md. Would you be willing to include that update in your PR?

@jeremylong
Copy link
Owner

documentation added via #5180. Messed up trying to update this PR - so I just created a new one.

@jeremylong jeremylong closed this Dec 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli changes to the cli
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants