[GitHub] [openmeetings] sebawagner commented on a change in pull request #111: OPENMEETINGS-2434 Add some logging to better find out what env variable has been picked up.

[GitBox]

sebawagner commented on a change in pull request #111:
URL: https://github.com/apache/openmeetings/pull/111#discussion_r487689889



##########
File path: 
openmeetings-util/src/main/java/org/apache/openmeetings/util/OmFileHelper.java
##########
@@ -104,6 +104,7 @@ public static void setOmHome(File home) {
                if (Strings.isEmpty(dataDir)) {
                        DATA_HOME = new File(OM_HOME, DATA_DIR);
                } else {
+                       log.info("Overwrite custom DATA_DIR: {}", dataDir);

Review comment:
       @solomax we could do that actually. Init DATA_DIR in Application.java, 
which is a Spring managed class. 
   
   And then when it calls OmFileHlper.setOmHome(File home, String dataHome)
   
   That would be actually quite trivial. And I think better than via env 
variables.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org