Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 321843

Summary:

session manager should handle errors from logout.

Product:

z_Archived

Reporter:

Vikram Bhatia <vikram.jeet.bhatia>

Component:

Eclipselink

Assignee:

David Minsky <david.minsky>

Status:

CLOSED FIXED

QA Contact:

Project Inbox <eclipselink.foundation-inbox>

Severity:

normal

Priority:

CC:

david.minsky, tom.ware, vikram.jeet.bhatia

Version:

unspecified

Target Milestone:

---

Hardware:

All

OS:

All

Whiteboard:

submitted_patch

Attachments:

Description	Flags
proposed fix	none
Proposed Fix	none
patch	david.minsky: iplog+

Description Vikram Bhatia

2010-08-05 05:59:48 EDT

Build Identifier: 

Session Manager should handle errors from session.logout(). It has been handled in SessionManager for session refresh, however it needs to be handled in destroy() method as well.

Reproducible: Couldn't Reproduce

Steps to Reproduce:
In case, some exception is thrown by session logout in SessionManager.destroy(), session is not being removed from SessionManager.

Comment 1 Vikram Bhatia

2010-08-05 06:01:53 EDT

Created attachment 175915 [details]
proposed fix

Comment 2 Vikram Bhatia

2010-08-05 06:13:58 EDT

Created attachment 175917 [details]
Proposed Fix

Comment 3 Vikram Bhatia

2010-08-05 12:40:14 EDT

Created attachment 175958 [details]
patch

Comment 4 Tom Ware

2010-08-09 14:02:57 EDT

Setting target and priority.  See the following page for details of the meanings of these fields:

http://wiki.eclipse.org/EclipseLink/Development/Bugs/Guidelines

Comment 5 David Minsky

2010-08-12 12:21:40 EDT

Checked in.
Fixed in r8030

Comment 6 Eclipse Webmaster

2022-06-09 10:04:57 EDT

The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink