Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 324107 - getEnvironment command for the Processess service does not comply with the protocol
Summary: getEnvironment command for the Processess service does not comply with the pr...
Status: RESOLVED FIXED
Alias: None
Product: TCF
Classification: Tools
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 0.4.0   Edit
Assignee: Project Inbox CLA
QA Contact: Martin Oberhuber CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-31 12:41 EDT by Sidharth Kodikal CLA
Modified: 2013-06-05 07:56 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sidharth Kodikal CLA 2010-08-31 12:41:44 EDT
Build Identifier: 

The protocol documented for the getEnvironment command in the Processes service requires that the environment be a JSON object

C • <token> • Processes • getEnvironment •
R • <token> • <error report> • <object: environment variables> •

As of version 1134 of trunk for the agent sources, the environment returned is instead a JSON array.

It looks like either the documentation or the implementation (agent and java proxy) need to be corrected.


Reproducible: Always

Steps to Reproduce:
I ran into this when writing a python client that speaks to the TCF C agent running on Linux. My client was expecting a JSON object per the documentation and failed the getEnvironment command because it did not get the expected reply.
Comment 1 Eugene Tarassov CLA 2010-08-31 13:17:25 EDT
Fixed. I have changed the docs to match implementation.
Thanks!
Comment 2 Sidharth Kodikal CLA 2010-09-02 16:23:44 EDT
(In reply to comment #1)
> Fixed. I have changed the docs to match implementation.
> Thanks!

Thanks. I have updated my client to use the changed protocol. As expected, it works for me now.
Comment 3 Doug Schaefer CLA 2011-05-17 10:51:02 EDT
Moving bugs to new home for IP log.
Comment 4 Martin Oberhuber CLA 2013-06-05 06:28:53 EDT
Bulk change: Marking all bugs from the TM era (until June 2011) target 0.3