The logging is a bit messy nowadays with all the different distros. We
could still use org.apache.commons.logging for simplicity and the use
log4j-jcl bridge to redirect to Log4j 2. Just need to have have a
log4j2.xml file (with the jars). Maybe I could do look at this to separate
the logging from the roller config?
Cheers Greg
On 31 August 2017 at 03:08, Dave <snoopd...@gmail.com> wrote:
> Thanks for taking a look Greg! I really appreciate it.
>
> I'll try to address those issues this weekend at the latest, and any other
> issues you uncover.
>
> I did look at moving to Log4j2 but it looked like too large of an effort
> (but mainly because of the way Roller expects Log4j properties to be in the
> Roller config file).
>
> Dave
>
>
> On Wed, Aug 30, 2017 at 4:12 AM Greg Huber <gregh3...@gmail.com> wrote:
>
> > Did a quick test from the war (will set up a dev environment so can do
> some
> > more testing!), here is what i found:
> >
> > Hard coded stuff :
> >
> > roller.properties
> >
> >
> > log4j.appender.roller.File=/Applications/Java/apache-
> tomcat-8.5.9/logs/roller.log
> >
> > (maybe switch to log4j2?
> > <Property name="log-path">${sys:catalina.home}/logs</Property>)
> >
> > ##
> >
> > Media upload gives this error (but works):
> >
> > SEVERE: Servlet.service() for servlet [jsp] threw exception
> > org.apache.jasper.JasperException:
> > /WEB-INF/jsps/editor/MediaFileAddSuccess.jsp (line: [83], column: [12])
> > Attribute [id] invalid for tag [iterator] according to TLD
> >
> > <s:iterator value="newImages" id="newImage">
> >
> > id="newFile" two of these!
> >
> > ###
> >
> > Menu preferences/maintenance
> >
> > Flush cache and Reset Hitcount
> >
> > >> Method flushCache for action maintenance is not allowed!
> >
> > Possibly strict-method-invocation="true" is on, so could try to add
> > allowed-methods to your struts.xml for the class
> >
> > <action name="maintenance"
> > Class="org.apache.roller.weblogger.ui.struts2.editor.Maintenance">
> > <result name="success" type="tiles">.Maintenance</result>
> > <allowed-methods>flushCache,reset</allowed-methods>
> > </action>
> >
> > (btw your previous dmi would have worked, so there was no need to change
> > it!)
> >
> > ##
> >
> > Cheers Greg
> >
> > On 29 August 2017 at 02:10, Dave <snoopd...@gmail.com> wrote:
> >
> > > I've prepared a second release candidate for Roller 5.2.0. The first RC
> > > depended on Struts 2.3 and this new RC-2 release depends on Struts
> > 2.5.10.1
> > > and no longer uses Dynamic Method Invocation (DMI). Because of this
> > change,
> > > a significant number of JSP pages were changed.
> > >
> > > I've been testing this RC-2 release locally and on rollerweblogger.org
> > > for the past week and so far it seems stable.
> > >
> > > Apart from the Struts upgrade, Roller 5.2.0 is primarily a bug fix
> > release,
> > > with lots of fixes from Kohei.
> > >
> > > You can find the release candidate here:
> > > https://dist.apache.org/repos/dist/dev/roller/roller-5.2/v5.
> 2.0-rc-2
> > >
> > > And the list of issues resolved is here:
> > >
> > > https://issues.apache.org/jira/secure/ReleaseNote.jspa?
> > > projectId=12310906&version=12340980
> > >
> > > Please take a look at the release and vote within the next 72 hours.
> > >
> > > My vote is +1 to release.
> > >
> > > Thanks,
> > > Dave
> > >
> >
>
