CVS log for pegasus/src/Pegasus/Server/CIMOperationResponseEncoder.cpp
	Help

Request diff between arbitrary revisions

BUG#: 9676
TITLE: Add pull operations to the OpenPegasusclient and server

DESCRIPTION: This patch is the implementation of the pull operations
defined in DSP0200 for both the CIM Client and CIM Server.  It includes
test programs in Pegasus/Client/tests and extends cimcli to make pull
requests and receive pull operation responses. There are still limitations
and issues noted in the readme.pulloperations file as well as some KS_TOD
comments marking things to do in the source code.

BUG#: 9676
TITLE: Remove a number of TODO comments left in the code

DESCRIPTION: Removed TODO comments and commented or some of the diagnostic
traces that were in the code from development. No functional changes

BUG#: 9676
TITLE: TASK-PEP317_pullop-branch  Issue with executing the pull opts
tests on some platforms

DESCRIPTION: Modified pullop Makefile to reduce max size of responses
in tests.  Also cleaned up a number of left over TODOs from the
development

BUG#: 9676
TITLE: TASK-PEP317_pullop-branch. New functionality and code cleanup

DESCRIPTION: OpenQueryInstances now working. Fixed issue with cimcli,
fixed UIntArgs to use smart pointers, Cleaned up CreateContext, Cleaned
up code in WQL/CQLOperationRequestDispatcher.cpp to create common
function, other minor.  See readme for 30 May for list.

BUG#: 9676
PEP#: DELETE ONE OF THESE LINES
TITLE: TASK-PEP317_pullop-branch. Multiple updates to


DESCRIPTION: Changed PullResponse... msg to OpenOrPullResponseDataMessage,
Added diagnostics to catch errant issue with EnumerationContextTable,
fixed issue in msg bundle, corrected issues with static tests and other
miscellaneous minor changes.

BUG#: 9676
TITLE: TASK-PEP317_pullop-branch. Minor changes

DESCRIPTION:  Extended tests. Minor changes to get better diagnostics
trace.

PEP#: 317
TITLE: TASK-PEP317_pullop-branch Merge Out from head of tree effective 21 Nov 2013.

DESCRIPTION:

PEP#: 317
TITLE: TASK-PEP317_pullop-branch - Add CIMOperationResponse
encoder for OpenQueryInstances and pullInstances

DESCRIPTION: Add responder that returns just instances in
CIMResponseData for OpenQueryInstances and pullInstances

PEP#: 317
TITLE: TASK-PEP317_pullop-branch pull operations, very minor changes
to better sync branch with head of tree

DESCRIPTION: Removed some unused code, etc.

PEP#: 317
TITLE: TASK PEP 317, Pull Operations

DESCRIPTION: Add OpenQuery, Clean up EnumContext

BUG#: 9999
TITLE: TASK_317-Pull Operations update.

DESCRIPTION: This update brings this branch back in line with the current
head of tree and cleans up a number of outstanding issues in the code.  Note
That there are still a lot of outstanding issues being worked on.

BUG#: 9628
TITLE: PEGASUS_ASSERT used in places where PEGASUS_UNREACHABLE should be used

DESCRIPTION:

BUG#: 5904
TITLE: Generates non-conformant xml for associators response

DESCRIPTION: Modify response paths to insure that only the correct
objectpath type (ClassPath or InstancePath) is generated and move
the hole decision path for class vs. instance requests back as close
to the client as possible so that the server uses operation request
information and not keybinding counts to determine if a cimobjectpath
is class or instance

BUG#: 99999
TITLE: Task PEP317 branch Pull ops

DESCRIPTION: Bring property list processing into line

BUG#: 99999
TITLE: Task317 Branch -- pull operations. Sync to head of tree

DESCRIPTION: Sync the branch back to the head of cvs tree

BUG#: 9999
TITLE: Merge out head to Task-PEP317 branch using Task Makefile

DESCRIPTION:  Update this working branch to Pegasus Head of Tree as of
about 8 Oct 2011

BUG#:8966
TITLE: Client hangs on local connection with binary enabled, non-chunking but multiple provider responses

DESCRIPTION:

BUG#:8961
TITLE: Client hangs on local connection with binary enabled.

DESCRIPTION:

BUG#:8544
TITLE: Add tracing for binary protocol data

DESCRIPTION:

BUG#: 8874
TITLE: Commit code for pull operations to branch TASK-PEP317_pullop-branch

DESCRIPTION: Commits the current code for the pull operations to a
task branch for further review and testing before it is committed to
the Pegasus head.

PEP#: 348
TITLE: Initial commit of SCMO for CMPI

DESCRIPTION: Initial commit of SCMO for CMPI

PEP#: 348
TITLE: Merge head into branch.

DESCRIPTION: Merge head into branch.

PEP#:348
TITLE: Building path through server for all data formats using the CIMResponseData class.

DESCRIPTION:

Open items:

Binary transformation for SCMO
Internal Xml transformation for SCMO
SCMOXmlWriter not yet complete

Other places are marked using TODO

PEP#:348
TITLE: Implementing CMPI_Instance and CMPI_ObjectPath through SCMO

DESCRIPTION: Initial implementation of CMPI for SCMO usage and scaffolding test code

BUG#: 8586
TITLE: The client side hang on when the server side fails to reallocate memory
DESCRIPTION:

BUG#: 8567
TITLE: The client side hang on when the server side fails to reallocate memory
DESCRIPTION: Server closes the connection and logs a message when out-of-memory error occurs during response encoding/aggregation or request handling.

BUG#:8393
TITLE: Extend On-demand de-serialization for Associators and ExecQuery

DESCRIPTION: Extend On-demand de-serialization for Associators and ExecQuery

BUG#: 8182
TITLE: Wbemexec handling of WMIMapper's localConnect functionality
DESCRIPTION: The Wbemexec is enhanced to handle the WMIMapper's localConnect functionality

BUG#: 8273
TITLE: Remove trailing space characters
DESCRIPTION: Remove meaningless whitespace.

BUG#: 8123
TITLE: Update copyright/license text

DESCRIPTION:

Fixing trailing spaces in copyright/license text

BUG#: 8123
TITLE: Update Coyright/License text

DESCRIPTION:

Adding the new copyright/license text to almost all files

BUG#: 8171
TITLE: Binary Protocol

DESCRIPTION: Initial commit of binary protocol feature.

BUG#: 8074
TITLE: Internal Binary Protocol

DESCRIPTION: Internal Binary Protocol

BUG#: 7830
TITLE: Logger::put_l should use MessageLoaderParms
DESCRIPTION: Use a MessageLoaderParms parameter in localizable logging calls.

BUG#: 7784
TITLE: Use char* instead of String for message IDs
DESCRIPTION: Do not convert message IDs to Strings.

PEP#: 328
TITLE: Solaris Porting

DESCRIPTION: Ongoing Solaris Porting

BUG#:7721
TITLE: Trace Level checking - first block - Pegasus/Server

DESCRIPTION:

BUG#:7645
TITLE: PEP#315 - Improve Trc&Log - Remove Logger::TRACE

DESCRIPTION:

PEP#: 311
TITLE: WS-Management Support in CIM Server.
DESCRIPTION: elements of SOAP encoder and writer.

BUG#: 7238
TITLE: Move thread language logic from Message to CIMMessage
DESCRIPTION: Clean up the Message class, move language logic to CIMMessage, and clean up the Thread::setLanguage logic.

BUG#: 6682
TITLE: Use MessageType enum instead of Uint32 for message types
DESCRIPTION: Use the strongly-typed MessageType enum for all message type variables.

BUG#:6492
TITLE: add missing method exit trace statements

DESCRIPTION:

BUG#:6300
TITLE: Performance - simplify internal Tracer interface and use the more efficent Trace macros (2.6)

DESCRIPTION:

BUG#:6207
TITLE: Performance - make OpenPegasus use its more efficent Trace macros

DESCRIPTION:

BUG#: 5947
TITLE: Pegasus support for 64-bit Windows platform

DESCRIPTION:

Add support for the WIN64_IA64_MSVC and WIN64_X86_64_MSVC platforms and remove warnings associated with 64-bit windows compilation.

BUG#: 4408
TITLE: HTTP trailer information does not show up in xmlIO trace output

DESCRIPTION: Committing approved fix

BUG#: 5823
TITLE: G11n Issues in CMPI interface

DESCRIPTION: Committing approved fix

BUG#: 5850
TITLE: Source files do not meet chksrc requirements
DESCRIPTION: Remove tab characters and author names and shorten lines that are wider than 80 characters.

BUG#:5605
TITLE: Compile failure with PEGASUS_DISABLE_PERFINST defined

DESCRIPTION: getTotalServerTime should only be called if PEGASUS_DISABLE_PERFINST is not defined.

BUG#: 5584
TITLE: Response chunks may be sent out of order
DESCRIPTION: Ensure the same thread is used to write response chunks from the Dispatcher through the ResponseEncoder and HTTPConnection.

BUG#: 5579
TITLE: Response chunks may be sent out of order
DESCRIPTION: Ensure the same thread is used to write response chunks from the Dispatcher through the ResponseEncoder and HTTPConnection.

BUG#: 5518
TITLE: Clean up statistics code in Message class
DESCRIPTION: Remove unused methods and simplify the logic in the statistics code.

PEP#: 233
TITLE: Updated Embedded Instance Support branch

DESCRIPTION:

- Merged with latest code from 2.5.1

- Modified ObjectNormalizer to use a CIMOMHandle instead of the repository and to normalize the instances in an embedded instance property.

BUG#: 4691
TITLE: Update Licenses to 2006

DESCRIPTION: Updates most of the licenses to 2006. The slp_client directories are excluded for the moment pending discussion. This change has passed unit and system tests.  Note that this changes just about EVERY file in Pegasus.

PEP#: 244
TITLE: Pegasus Server Profile support

DESCRIPTION:

- Merge branch with latest changes from trunk
- Cache repository info in Interop provider for much faster response times during ElementConformsToProfile operations

BUG#: 4644
TITLE: ContentLanguages and AcceptLanguages classes should be named ContentLanguageList and AcceptLanguageList
DESCRIPTION: Rename the ContentLanguages and AcceptLanguages classes to ContentLanguageList and AcceptLanguageList to indicate that they are not specific to the HTTP protocol.

BUG#: 4506
TITLE: Tracing/Logging performance

DESCRIPTION: Removed gratuitous construction of temporary string object assocition with tracing and logging.

BUG#: 4442
TITLE: Changed server response time variable to be Uint64.

DESCRIPTION:

BUG#: 4451
TITLE: Compile failure building CIMOperationResponseEncode.cpp

DESCRIPTION: Changed type from Uint32 to Uint64.

BUG#: 4307
TITLE: Buffer class optimization

DESCRIPTION: New Buffer class and usage.

BUG#: 4200
TITLE: Performance Optimizations

DESCRIPTION: Replaced internal uses of Array<char> with Buffer.

BUG#: 2290
TITLE: pegasus should honor HTTP header  'Connection: Close'


DESCRIPTION:
  Modified code for both the Server and the Client to
  handle a "Connection: Close" header.

BUG#: 3935
TITLE: cimserver hang waiting for mutex

DESCRIPTION: This is a work-around for a C-Cover problem. There appears to be a problem associated with the use of the cov_write call in a multi-threaded environment.  Since we are now able to shutdown the cimserver successfully, the calls to cov_write are not longer necessary and can be removed.

BUG#: 3211

TITLE:Client Performance Statistics Test Fails

DESCRIPTION: checking in code to fix test case and checking in code to move statisitcs gathering from CIMDateTime based to TimeValue based

PEP#: 128

TITLE:Client Statisitcal gathering

DESCRIPTION: the server time was not being retruned to the client correctly . This change fixes that problem

BUG#: 2656
TITLE: License statement update to 2005

DESCRIPTION: All .h, .cpp and the wql, cql, compile *.l and *.y files along
with the .java files in providermanager2/jmpi updated to use the license
statement in doc/license.txt
Tested on windows and visual test of all diffs.

BUG#: 2297
TITLE: 'Types' changes

DESCRIPTION: Changes for typedef'ing Sint8 to be a 'signed char'.

BUG#: 2231
TITLE: Globalization test fails

DESCRIPTION: now fixed for 2.5 with the blessing of chuck carmack

BUG#: 2230
TITLE: TestClient failing because of timeout.
DESCRIPTION: This fix is involved and fixes many other bugs that are all related. See comments in the bug

BUG#: 2196
TITLE: Copyright update

DESCRIPTION: Update all .cpp and .h files for new license and
update the doc/license.txt file.  Note that there were also
a couple of files that had to be fixed because they had violated
the comments rules (ex. blank line at head of file or in the case of
xmlwriter.cpp a comment line //=========  which drove the strip
function nuts.  These were fixed.  This has been compiled and tested
on windows.

BUG#: 1686
TITLE: Chunking does not seem to handle server errors correctly

DESCRIPTION: Removed a cerr print out

BUG#: 1686
TITLE: Chunking doesnt seem to handle server errors correctly
DESCRIPTION:

problem:
a. The full error description was not coming back to client.
b. Content language was not being communicated to client on an error condition.

solution:
a. call TraceableCIMException().getDescription() to expand the error text.
   When chunking was implemented, there was a call that originally placed the
   error in the XML payload, but the cim operations spec dictates that the XML
   payload is to be ignored on error conditions, however, the full error still
   needs to be placed into the CIMStatusCodeDescription field in the HTTP
   trailer to communicate the error outside of the XML payload.
b. call setContentLanguages() on the cim exception object

files fixed:
a. Server/CIMOperationResponseEncoder.cpp
b. Client/CIMOperationResponseDecoder.cpp

BUG#: 1633
TITLE: HTTP response has incomplete XML

DESCRIPTION:

The order of the parameters to the various XMLWriter::format*RspMessage functions is incorrect, causing the serverResponseTime to be interpreted as the isLast parameter, sometimes being cast to "false." Fixing the parameter order should fix this bug.

PEP#: 128

TITLE: Performance statistics sent to CIM Client

DESCRIPTION: add code to suppot WBEMServerResponseTime in headers

PEP#:140
TITLE:phase1 of HTTP chunked transfer encoding support

DESCRIPTION:
- phase1 only impacts the HTTP connection level and the xml writer and its callers.
- phase1 does NOT change the flow within the server to be async
- Hooks have been placed in Message.h for the eventual support in phase2.
- phase2 will eventually change ResponseHandler::deliver() to be async
- servers that send chunked encoding can now be parsed by the pegasus client
- all changed files for phase1 are:

Common/Message.h
Common/HTTPConnection.cpp
Common/HTTPConnection.h
Common/HTTPMessage.cpp
Common/HTTPMessage.h
Common/XmlWriter.cpp
Common/XmlWriter.h
Server/CIMOperationResponseEncoder.cpp
Server/CIMOperationResponseEncoder.h
Client/CIMOperationResponseDecoder.cpp
Client/CIMClientRep.cpp

PEP#: 135
TITLE: OperationContext Enhancements

DESCRIPTION:
Please refer to PEP135 for details.

PEP#: 130
TITLE: Remote CMPI

DESCRIPTION: Mistakenly comitted modified version of CIMOperationResponseEncoder.cpp

PEP#: 130
TITLE: Remote CMPI support

DESCRIPTION: Base code for Remote CMPI

PEP#:101
TITLE: AutoPtr enhancements
DESCRIPTION: Plz refer PEP101 for details.

Bug 1145: OS/400 Specific Changes

HP-[DME] Bugzilla 989 - Added DiscardedData trace entry.

PEP 55 Update license on source files to current license text and date

IBM-CC, Bug 517, R2.3: Response logging reads past the end

IBM-CC, Bug 517, R2.3: Response logging reads past end of array

IBM-CC, Bug 517, R2.3: Response logging reads past end of array

merge with 2.3 localized code

Bug 419: OS/400 Authentication Enhancements

IBM:HR,PEP58,R2.3:Globalization Changes

g10n changes merged

IBM-CC, PEP58, R2.3 - Globalization changes

IBM:DR [PEP:53] Logging Enhancements Code Points

HP-SF: Fixed a problem that was causing cimserver to lose a request when it was running out of memory. It has now been fixed to catch the exception and send an appropriate response.

HP-[CAKG] respond correctly to POST requests

HP-[CAKG] API Review: CIMName modifications

HP-RK API Review: Change String::allocateCString() to getCString().

HP-RK Update license header.

converted some classes back to MessageQueue from MessageQueueService in order to reduce idle threads.

HP-RK Remove purify instrumentation.

HP-RK Moved CIM*::toXml() methods to XmlWriter::append*Element().  Moved CIM*::print() methods to XmlWriter::print*Element().  Putting this function in XmlWriter instead of the CIM* classes keeps those internal methods out of the published client and provider APIs.

HP-RK Moved CIMClass::toXml() method to XmlWriter::appendClassElement().  Moved CIMInstance::toXml() method to XmlWriter::appendInstanceElement().  Similar for print() methods.

HP-RK Moved CIMValue::toXml() method to XmlWriter::appendValueElement().

HP-DME: Modifications to support NULL qualifier values.

HP-[Bapu Patil] Added PURIFY build environment.

Performance measurement instrumentation

HP-RK Changed the MessageQueue::getQueueName() method from virtual to non-virtual, and removed that method definition from all the subclasses.  To make sure that the MessageQueue class gets the queue name value, the default value for the queue name parameter to the MessageQueue constructor was removed.  Constants were defined in Constants.h for all the queue names currently used in Pegasus.  The places where those queue names were being used for lookup were also changed to use the constants.

HP-RK Fixed defects in InvokeMethod responses.  Specifically, made the RETURNVALUE XML tag optional and propagated the return value from the provider to the response encoder.

HP-RK Modified the CIMOperationResponseMessage class and its subclasses to contain a CIMException instead of a CIMStatusCode and String description.  This allows extra information in a CIMException, such as file name and line number, to be carried internally throughout the system.

HP-RK Add some support for ExecQuery in the client.

HP-RK Add some support for ExecQuery in the server.  Clean up the client response that indicates the operation is unsupported.

HP-RK Define a common location (Common/Constants.h) for Pegasus server constants.  Define an initial set of these constants.

change toXml function

HP-RK Consolidate formatting of error responses in XmlWriter.

HP-RK Massive changes in XmlWriter to eliminate redundant memory copies.  Also lots of cleanup.

 lots of changes to support mixing of legacy and async messages in queues

fixed problem with MessageQueueService::_enqueueResponse handling of legacy messages

HP-RK Recognize and generate the PARAMTYPE attribute to RETURNVALUE to communicate the type of the return value for an extrinsic method.  Add the ability for a CIMValue to contain an array of references.

HP-RK Recognize and generate the PARAMTYPE attribute to PARAMVALUE to communicate the types of extrinsic method input/output parameters.  Clean up the CIMParamValue class to remove unnecessary data and methods.

MessageQueueService integration

Add additional trace code.

experimentation with MessageQueue

Merged dev branch into main trunk.

FIX: Linux build needs Pegasus/Common/Config.h included first

async dispatcher updates

HP-RK Add ATTN to handle encoding of non-String InvokeMethod parameters.

HP-RK Changed the return value of EnumerateInstances from Array<CIMInstance> to Array<CIMNamedInstance>.  Changed the parameter to ModifyInstance from CIMInstance to CIMNamedInstance.  Added an optional PropertyList parameter to ModifyInstance.  Changed all handling of PropertyList parameters to use the CIMPropertyList class instead of Array<String>.  Each of these changes was required to comply with the DMTF specification.  These changes were made throughout the system except for in the provider API.  For now, the CIMProviderHandle and CIMBaseProviderHandle hide the differences from providers, so older providers will continue to function.  However, this means that clients can not yet take advantage of the added functionality.

HP-RK Added a toXml(void) method to CIMValue.  Eliminated improper use of CIMValue.toString(), often replacing with CIMValue.get(String&) or CIMValue.toXml().  Added some error handling in the indication handlers.

tracer big fixed

HP-RK Implement encoding/decoding of GetPropery/SetProperty/EnumerateInstances request/response messages.

HP-YZ Fixed createInstance to return a CIMReference (instanceName).

HP-SF Added an interface to the Tracer to allow tracing a specific number of bytes in a string. Modified the XmlIO trace code to use the new interface. Changed the config file defaults for tracing.

HP-[Nitin Upasani] Implementation of InvokeMethod.

HP-DE Minor changes to support code coverage build.

HP-YZ Added tracing in files CIMOperationRequestDecoder.cpp and CIMOperationResponseEncoder.cpp so XML input and output can be written in a trace file.

new

file CIMOperationResponseEncoder.cpp was initially added on branch dev.