java-compat from MVM's - what should I set to read logging.properties ?? #124

Activity

1. lesv

   added

   bug

   on Nov 10, 2015
2. erugeri commented on Nov 10, 2015

   

   erugeri

   on Nov 10, 2015

   More actions

   In my appengine-web.xml I have:

   <system-properties>
       <property name="java.util.logging.config.file" value="WEB-INF/logging.properties"/>
   </system-properties>
   

   To check if it works, set the content of logging.properties like that:

   .level=ALL
   
3. lesv commented on Nov 10, 2015

   

   lesv

   on Nov 10, 2015

   Author

   More actions

   I'm seeing file not found (in the log) for all three variations.
4. gregw

   self-assigned this

   on Dec 16, 2015
5. gregw commented on Dec 16, 2015

   

   gregw

   on Dec 16, 2015

   Contributor

   More actions

   @lesv
   I've found, fixed and forgotten this bug once already. I'll refresh my brain and refix shortly.
6. gregw commented on Jan 8, 2016

   

   gregw

   on Jan 8, 2016

   Contributor

   More actions

   Depending on your environment, PR #148 may have only partially fixed this issue. The fix in that PR was to prevent the system property from being masked.

   However, there is still an issue with the assumption that the working directory is such that a relative path to WEB-INF/logging.properties will correctly resolve.

   A fix to the relative path issue has been merged to async-support branch with #150, as the VmRuntimeWebAppContext now looks for a log directory relative to WEB-INF and converts it to an absolute path.

   I will close this issue for now - but if you want the fix back ported to master branch prior to the async-support merge, please reopen.
7. gregw

   closed this as completedon Jan 8, 2016
8. meltsufin

   reopened this on Mar 1, 2017
9. meltsufin commented on Mar 1, 2017

   

   meltsufin

   on Mar 1, 2017

   Contributor

   More actions

   We need this fixed to be merged into master. However, it looks like it works with Jetty 9.3.5, just not with higher versions.