Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 321843 - session manager should handle errors from logout.
Summary: session manager should handle errors from logout.
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Eclipselink (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: David Minsky CLA
QA Contact: Project Inbox CLA
URL:
Whiteboard: submitted_patch
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-05 05:59 EDT by Vikram Bhatia CLA
Modified: 2022-06-09 10:04 EDT (History)
3 users (show)

See Also:

Attachments
proposed fix (959 bytes, patch)
2010-08-05 06:01 EDT, Vikram Bhatia CLA 		no flags Details | Diff
Proposed Fix (1.27 KB, patch)
2010-08-05 06:13 EDT, Vikram Bhatia CLA 		no flags Details | Diff
patch (975 bytes, patch)
2010-08-05 12:40 EDT, Vikram Bhatia CLA 		david.minsky: iplog+
Details | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Vikram Bhatia CLA 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 CLA 2010-08-05 06:01:53 EDT 
Created attachment 175915 [details]
proposed fix
Comment 2 Vikram Bhatia CLA 2010-08-05 06:13:58 EDT 
Created attachment 175917 [details]
Proposed Fix
Comment 3 Vikram Bhatia CLA 2010-08-05 12:40:14 EDT 
Created attachment 175958 [details]
patch
Comment 4 Tom Ware CLA 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 CLA 2010-08-12 12:21:40 EDT 
Checked in.
Fixed in r8030
Comment 6 Eclipse Webmaster CLA 2022-06-09 10:04:57 EDT 
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink