jfrog / build-info

Artifactory's open integration layer for CI build servers
https://www.buildinfo.org
Apache License 2.0
145 stars 154 forks source link

Client configuration should prefer system properties #775

Closed yahavi closed 6 months ago

yahavi commented 6 months ago

Fix https://github.com/jfrog/jfrog-cli/issues/2373

The setup for the client should prioritize Java system properties over the default properties set by the JFrog CLI. Take this command, for instance, where the JFrog CLI specifies artifactory.publish.artifacts=true, but a conflicting false value is set by a system property:

jf mvn clean install -Dartifactory.publish.artifacts=false

In this case, the system property should take precedence over the default value from the JFrog CLI.

github-actions[bot] commented 6 months ago
[![👍 Frogbot scanned this pull request and found that it did not add vulnerable dependencies.](https://raw.githubusercontent.com/jfrog/frogbot/master/resources/v2/noVulnerabilityBannerPR.png)](https://docs.jfrog-applications.jfrog.io/jfrog-applications/frogbot)

[🐸 JFrog Frogbot](https://docs.jfrog-applications.jfrog.io/jfrog-applications/frogbot)