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
AMBARI-24833. Create cloud input/output skeleton. #17
Conversation
ambari-logsearch-logfeeder/src/main/java/org/apache/ambari/logfeeder/conf/LogFeederMode.java
Outdated
Show resolved
Hide resolved
ambari-logsearch-logfeeder/src/main/java/org/apache/ambari/logfeeder/conf/LogFeederProps.java
Outdated
Show resolved
Hide resolved
ambari-logsearch-logfeeder/src/main/java/org/apache/ambari/logfeeder/conf/LogFeederProps.java
Outdated
Show resolved
Hide resolved
...java/org/apache/ambari/logfeeder/manager/operations/impl/CloudStorageInputConfigHandler.java
Show resolved
Hide resolved
...main/java/org/apache/ambari/logfeeder/manager/operations/impl/DefaultInputConfigHandler.java
Show resolved
Hide resolved
...main/java/org/apache/ambari/logfeeder/manager/operations/impl/DefaultInputConfigHandler.java
Show resolved
Hide resolved
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
FileUtils.writeStringToFile(pwdFile, defaultSecret, Charset.defaultCharset()); | ||
return defaultSecret.toCharArray(); | ||
} else { | ||
return FileUtils.readFileToString(pwdFile, Charset.defaultCharset()).toCharArray(); |
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.
File content can be read directly to byte or char array That would be more secure than readFileToString. Also defaultSecret can be a char[]
* AMBARI-24833. Create cloud input/output skeleton. * AMBARI-24833. Use LogFeederMode as enum by spring * AMBARI-24833. Fix review issues + add secret stores.
What changes were proposed in this pull request?
both mode can work independently from each other (see the Conditional beans in ApplicationConfig)
How was this patch tested?
with docker env by changing the default cloud mode