-
Notifications
You must be signed in to change notification settings - Fork 472
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
remoteBackup: fix invalid config file breaks Artifactory instance #349
remoteBackup: fix invalid config file breaks Artifactory instance #349
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ |
be393af
to
c0a14a6
Compare
I have read the CLA Document and I hereby sign the CLA |
c0a14a6
to
be393af
Compare
|
||
// Ensure invalid json config does not prevent Artifactory | ||
// from serving and receiving artifacts. | ||
if (!cfgfile.exists()) return |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A log message for this event would be nice.
be393af
to
af9c02d
Compare
af9c02d
to
e01bd91
Compare
@DarthFennec Can you please provide a review? |
1e443b4
to
e01bd91
Compare
@diepfote Are you still looking for this change? Its been open for long time. |
@yashprit-jfrog we integrated this internally. Feel free to close |
Closing based on above comment |
No description provided.