RE: Asking again: No registered context in security getCurrentContext() after apps run for two days.

2007-10-11 Thread Scott Stanchfield
Cool. Sounds even better. Prolly need to separate (in ExpirationMonitor) if (session.isDone() || age > maxLoginDurationMillis) { list.add(session); // to unregister session.setDone(true); it.remove(); } into if (session.isDone()) { list.add(session); /

Re: Asking again: No registered context in security getCurrentContext() after apps run for two days.

2007-10-10 Thread Scott Stanchfield
Sorry -- forgot the code... Replace assert context != null : "No registered context"; with if (context == null) throw new AccessControlException("No registered context (may have expired)"); Scott Stanchfield wrote: > > Suggested Fix > > Change

Re: Asking again: No registered context in security getCurrentContext() after apps run for two days.

2007-10-10 Thread Scott Stanchfield
We had the same issue, and didn't see much about it online (except this note and bug GERONIMO-2100) I debugged through the Geronimo code for a while today, and I think I've found the solution (which I also think is the proper solution to GERONIMO-2100 as well...) The problem [In this descriptio