percussion / percussioncms

Percussion CMS - Content Management System
https://percussioncmshelp.intsof.com/percussioncms
Apache License 2.0
11 stars 9 forks source link

Temp directory used by publishing is not using the value set in java.io.tmpdir #1118

Closed ramdev1980 closed 11 months ago

ramdev1980 commented 11 months ago

Describe the bug

Temp directory, used by publishing, is not using the value set in java.io.tmpdir. It should be picked up from system.properties. And it should be configured in server-beans.xml.

To Reproduce Temp directory value should be picked from java.io.tmpdir.

Expected behavior

Additional context

  1. For reference PSLocalDeliveryManager.java and server-beans.xml shuould be used.
  2. It should be configred like the below: <bean id="sys_deliveryManager" class="com.percussion.rx.delivery.impl.PSLocalDeliveryManager"> #{ systemProperties['java.io.tmpdir'] }/cms-temp/publish

natechadwick commented 11 months ago

We should also default the java.io.tmpdir in jettydefaults/start.d/jvm.ini to be the installation dir/temp folder that way on upgrade customers who do not override that property in jetty/base/start.d/jvm.ini will have no change in functionality.

natechadwick commented 11 months ago

Partially fixed by #1124