Hi Bram, you're right: it should not be logged as error. But IMHO LOG_INFO would be better than LOG_DEBUG since this is a valuable information, not only at debug-time. WDYT?
I will look into that as soon as I have some spare time, hopefully next weekend. If you need a fix sooner, feel free to change the log-level yourself ... after all OPS4J stands for 'open participation' ;-) Cheers, Matthias On Thu, Nov 25, 2010 at 9:56 AM, Bram de Kruijff (JIRA) <[email protected]> wrote: > Logging at LOG_ERROR when there is no Event Admin service > ---------------------------------------------------------- > > Key: PAXUSERADMIN-6 > URL: http://issues.ops4j.org/browse/PAXUSERADMIN-6 > Project: Pax User Admin > Issue Type: Bug > Components: Service > Affects Versions: 0.0.1 > Reporter: Bram de Kruijff > Assignee: Matthias Küspert > Priority: Minor > > > When there is no Event Admin service a message is logged at level LOG_ERROR. > This should be lowered as the specification states that EventAdmin must be an > optional dependency for any UserAdmin implementation (see 107.5.1): "User > admin events must be delivered asynchronously to the Event Admin service by > the implementation, if present.". I think LOG_DEBUG would be more appropriate. > > Originally reported at the Amdatu jira: > http://jira.amdatu.org/jira/browse/AMDATU-194 > > Sample snippet from my CI log: > build 24-Nov-2010 17:24:54 ERROR: > [org.ops4j.pax.useradmin.service.internal.UserAdminImpl] No event service > available - cannot send event of type 'ROLE_CREATED' for role 'Administrator' > [org.ops4j.pax.useradmin.pax-useradmin-service] > build 24-Nov-2010 17:24:54 ERROR: > [org.ops4j.pax.useradmin.service.internal.UserAdminImpl] No event service > available - cannot send event of type 'ROLE_CHANGED' for role 'Administrator' > [org.ops4j.pax.useradmin.pax-useradmin-service] > > > -- > This message is automatically generated by JIRA. > - > If you think it was sent incorrectly contact one of the administrators: > http://issues.ops4j.org/secure/Administrators.jspa > - > For more information on JIRA, see: http://www.atlassian.com/software/jira > > > _______________________________________________ general mailing list [email protected] http://lists.ops4j.org/mailman/listinfo/general
