[ https://issues.jboss.org/browse/AS7-1412?page=com.atlassian.jira.plugin.s... ] Thomas Diesler commented on AS7-1412: ------------------------------------- The ServerEnvironment.getModulesDir() supports a single directory only. IHMO its a bug ServerEnvironment that it allows invalid or non-existent directory configurations. -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/AS7-1412?page=com.atlassian.jira.plugin.s... ] Scott Marlow commented on AS7-1412: ----------------------------------- I think that ServerEnvironment.getModulesDir(), should return File[]. This would be more like ServerEnvironment.getJavaExtDirs(). -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/AS7-1412?page=com.atlassian.jira.plugin.s... ] Brian Stansberry updated AS7-1412: ---------------------------------- Fix Version/s: 7.1.1.Final Note that this issue is about validating inputs, not about exposing the module path. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/AS7-1412?page=com.atlassian.jira.plugin.s... ] Kabir Khan commented on AS7-1412: --------------------------------- I'm not really sure what needs doing here. Some wip lives here but I don't know if that is what is needed -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/AS7-1412?page=com.atlassian.jira.plugin.s... ] Brian Stansberry commented on AS7-1412: --------------------------------------- This is unfortunately probably a case by case thing, since there are a variety of flavors of wrongness. The home dir: if it doesn't exist or isn't a dir, fail. The modules dir: See the javadoc related to this sysprop/getter -- this is a meaningless thing. So, we should only fail if the user sets the sysprop and it doesn't exist. Since setting the prop indicates the user had some intent that has gone wrong. We shouldn't fail just because there's no "modules" dir under the home dir. The configuration dir must exist and must be a directory (although we need to be careful in ServerEnvironment in a managed domain, where the configuration dir does not need to exist.) The bundles dir is IMO like the modules dir. If people don't use OSGi and decide to delete that dir, we shouldn't fail here. OSGi can check it and fail there. But we should fail if they set the system property and the dir doesn't exist, as that implies intent. The data/ and tmp/ dirs we should probably check for the existence of the file. If it exists, fail if it isn't a directory. Otherwise, try and create it and fail if we can't. We know later on we are going to try and create these, so might as well do it now. The log dir is more like modules/bundles, since whether the dir is needed depends on the logging config. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/AS7-1412?page=com.atlassian.jira.plugin.s... ] Kabir Khan updated AS7-1412: ---------------------------- Fix Version/s: 7.1.1.Final (was: 7.1.2.Final) Git Pull Request: https://github.com/jbossas/jboss-as/pull/1620 (was: https://github.com/jbossas/jboss-as/pull/1620) -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira