danwatford commented on pull request #251: URL: https://github.com/apache/ofbiz-framework/pull/251#issuecomment-760898560
Hi @ieugen , I've been able to successfully test overriding properties by placing files in the dist's config directory. I tested in an openjdk-8 container using: `docker run -it --rm --publish 8080:8080 --publish 8443:8443 --mount type=bind,source=$PWD/build/distributions,destination=/distributions openjdk:8 bash -c "tar xf /distributions/ofbiz.tar; cd /ofbiz; bash"` I placed a copy of security.properties file in /ofbiz/config and set `password.length.min=10` Data was loaded with `bin/ofbiz -l` and then ofbiz started with `bin/ofbiz` I attempted to change the admin user's password to a single character value and was correctly shown an error message stating minimum password length was 10 characters, proving that the config override had worked. I then repeated the test using security.properties in windows which also passed. It seems url.properties and start.properties were not good candidates for overriding, perhaps they are loaded in an unusual way. ---------------------------------------------------------------- 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: [email protected]
