appengine-java-vm-runtime icon indicating copy to clipboard operation
appengine-java-vm-runtime copied to clipboard

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

Open lesv opened this issue 9 years ago • 5 comments

I've tried: <property name="java.util.logging.config.file" value="logging.properties"/> and <property name="java.util.logging.config.file" value="WEB_INF/logging.properties"/> and <property name="java.util.logging.config.file" value="/app/WEB_INF/logging.properties"/>

lesv avatar Nov 10 '15 01:11 lesv

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

erugeri avatar Nov 10 '15 08:11 erugeri

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

lesv avatar Nov 10 '15 15:11 lesv

@lesv I've found, fixed and forgotten this bug once already. I'll refresh my brain and refix shortly.

gregw avatar Dec 16 '15 23:12 gregw

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.

gregw avatar Jan 08 '16 00:01 gregw

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.

meltsufin avatar Mar 01 '17 23:03 meltsufin