[MAGNOLIA-1390] PropertyInitializer should not create a webapp/logs directory Created: 18/Feb/07  Updated: 18/Feb/07  Resolved: 18/Feb/07

Status: Closed
Project: Magnolia
Component/s: None
Affects Version/s: 3.0.1
Fix Version/s: 3.1 M1

Type: Bug Priority: Trivial
Reporter: Fabrizio Giustina Assignee: Fabrizio Giustina
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled

 Description   

PropertyInitializer always create a logs directory inside the webapp, regardless of the real directory used in log4j configuration. Also, starting from log4j 1.2.12, directories are created automatically by the file appenders so there is no need for creating them manually.



 Comments   
Comment by Fabrizio Giustina [ 18/Feb/07 ]

Fixed in trunk.
Also webapp/tmp should not be created since it's already created when calling when Path.getTempDirectory().

As a side note, the location where PropertyInitializer was trying to create dirs was also broken on windows:
DEBUG rootPath is D:/apps/something/webapp, webapp is webapp
DEBUG Creating C:\WINDOWS\system32\webapp\logs folder
DEBUG Creating C:\WINDOWS\system32\webapp\tmp folder

Generated at Mon Feb 12 03:26:27 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.