Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 327686 - Profile on server using JVMTI fails to attach to agents if AC has security enabled
Summary: Profile on server using JVMTI fails to attach to agents if AC has security en...
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: TPTP (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P2 major (vote)
Target Milestone: ---   Edit
Assignee: Mike Reid CLA
QA Contact: Kathy Chan CLA
URL:
Whiteboard: adopter patch
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-13 12:31 EDT by Mike Reid CLA
Modified: 2016-05-05 11:09 EDT (History)
2 users (show)

See Also:
jgwest: review+

Attachments
Patch (1.94 KB, patch)
2010-10-13 12:34 EDT, Mike Reid CLA 		no flags Details | Diff
Updated patch includes version number bump (2.69 KB, patch)
2010-10-13 16:17 EDT, Mike Reid CLA 		no flags Details | Diff
Show Obsolete (1) 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 Mike Reid CLA 2010-10-13 12:31:41 EDT 
The WTP Profile on Server integration operates by listening to 'server started' events and then if the server is running profiling mode a list of running agents is queried and attached to (or the user is prompted for specifics if necessary).

However, if the profiling agent is a JVMTI agent and the AC is secured, the attempt to list and attach to JVMTI agents fails. The result is that no agents are attached even though the server is listed as "Profiling".
Comment 1 Mike Reid CLA 2010-10-13 12:34:39 EDT 
Created attachment 180789 [details]
Patch

Attaching a patch which uses a security-aware API to connect to the AC to list JVMTI agents.

The user will be prompted for the username/password if their credentials have not already been cached.
Comment 2 Mike Reid CLA 2010-10-13 15:29:41 EDT 
Jon, can you please review this patch?
Comment 3 Jonathan West CLA 2010-10-13 15:46:12 EDT 
Patch is good.
Comment 4 Mike Reid CLA 2010-10-13 16:17:36 EDT 
Created attachment 180823 [details]
Updated patch includes version number bump
Comment 5 Mike Reid CLA 2010-10-14 09:39:29 EDT 
Patch checked into HEAD, resolving.
Comment 6 Kathy Chan CLA 2011-02-11 13:45:46 EST 
This defect had been resolved as FIXED for more than 1 month.  Please verify with the latest TPTP 4.7.2 driver.  If this defect is still left unverified by February 25, we'll close it on the originator's behalf.

TPTP 4.7.2 driver can be downloaded from:

http://www.eclipse.org/tptp/home/downloads/?ver=4.7.2
Comment 7 Mike Reid CLA 2011-02-11 14:48:08 EST 
Verified in TPTP-4.7.2-201102102100.
Comment 8 Mike Reid CLA 2011-02-11 14:49:34 EST 
Closing.