CVS log for pegasus/src/Pegasus/Client/CIMOperationResponseDecoder.cpp
	Help

Request diff between arbitrary revisions

BUG#: 9892
TITLE: Implement cookie-based session management.

DESCRIPTION: PAM (and GSSAPI) are quite slow when doing authentication.
Therefore cookie-based authentication is implemented. It is off by default
and it must be explicitly turned on by setting httpSessionTimeout
configuration option to nozero value.

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: 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
TITLE: TASK-PEP317_pullop-branch. Correct error noted in  bug9676,
Clean up OpenQueryInstances, error in cimcli handling of FQL
language parameter results in reject of request.

DESCRIPTION: Finished code for OpenQueryInstances to create common path
for ExecQuery and OpenQueryInstances. ExecQuery tested. fixed issue
in cimcli.  Corrected erroneous inclusion of execinfo.h header include,
removed more diagnostic messages, modified CreateContext function
interface.

BUG#: 9676
TITLE: TASK-PEP317_pullop-branch. Remove a number of diagnostics and clean
others

DESCRIPTION: About all diagnostics we will want to remove before final
checkin will be found by searching KS_TODO.  Also some commented code we
will remove has 4 / and a space to make it easy to find (//// ). Cleaned
up the enumContext unittest code so it compiles and runs.

PEP#:317
TITLE: TASK-PEP317_pullop-branch. CIMClient.h documentation and
some interprocess issues between timer thread and operation threads


DESCRIPTION: Added documentation for remaining functions in CIMClient.h
. Add protection for thread issues between timing thread and operation
threads. Clean up several TODOs in Cliient Response Doceder.

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 Extend processing Path for PullInstances

DESCRIPTION: Add code for PullInstances which is the pull component for the
OpenQueryInstances enumeration sequence.

PEP#: 317
TITLE: TASK-PEP317_pullop-branch pull operations, Clean up CIMMessage

DESCRIPTION: Clean up some code in CIMMessage.cpp and CIMMessage.h. This
moved some of the inheritance for the OpenResponse Classes and made
the constructors with optional parameters.   Moved the constructors to
CIMMessage.cpp

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#: 9564
TITLE: [f2fCleanUp] Remove build option PEGASUS_CLIENT_TRACE_ENABLE

DESCRIPTION:

BUG#: 9630
TITLE: use PEGASUS_UNREACHABLE in src/Pegasus/Client

DESCRIPTION:

BUG#: 9632
TITLE: Client trace error when trying to log

DESCRIPTION: Rewrote function by creating ClientTrace class in CIMClientRep
and doing all logic within that class

BUG#: 9520
TITLE:New warnings from clang 3.2 for unused private data``

DESCRIPTION:

BUG#:8987
TITLE: Get Rid of old PEGASUS_SNIA_INTEROP_TEST #ifdefs

DESCRIPTION:

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#:8911
TITLE: Nightly build failure due to "Bus error" on running CQL/tests/QueryContext

DESCRIPTION:

BUG#:8909
TITLE: Enforce binary communication between ProviderAgents

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.

BUG#: 8892
TITLE:D
cument and minor code redo for CIMResponseData set Binary function

DESCRIPTION: Changes to create separate function for
setRemainingBinary and document functions in CIMBuffer and remove
unused includes for BinaryCodec

BUG#: 8812
TITLE: CIM operation in provider using its cimom handle fails after idle connection timeout.

BUG#:8642
TITLE: New warnings reported by gcc 4.3

DESCRIPTION:

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: Extend just-in-time resolution in cim client

DESCRIPTION: Enable just-in-time resolution for an, r and rn operations

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

BUG#: 8605
TITLE: CIM operation fails after idle connection timeout due to server closing the connection.
DESCRIPTION: updated CIMClientRep to check connection status for every request and reconnect in the event of idle timeout.

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

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

BUG#:8400
TITLE: Delay resolution of binary data in client

DESCRIPTION: Delay resolution of binary data in client

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

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

BUG#: 8278
TITLE: HTTP headers are unnecessarily converted to Strings
DESCRIPTION: Process HTTP header names and values as character strings where feasible.

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

BUG#: 2418
TITLE: Questionable casting of getData() return values
DESCRIPTION: Do not cast away constness where it is not required.

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#: 7779
TITLE: Variables hidden by other declarations in scope
DESCRIPTION: Use different names for variables in nested scope.

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

DESCRIPTION:

BUG#: 7560
TITLE: CIMClient does not validate Content-Type charset
DESCRIPTION: Move the Content-Type validation from the HTTPMessage to the decoders, and make the validation in the CIMClient decoder consistent with the others.

BUG#: 7154
TITLE: Add timeout for idle connections (2.5 branch)

DESCRIPTION: Committing approved fix

BUG#: 7153
TITLE: Add timeout for idle connections (2.6 branch)

DESCRIPTION: Committing approved fix

BUG#: 7152
TITLE: Add timeout for idle connections (2.7 branch)

DESCRIPTION: committing approved fix

BUG#: 7143
TITLE: Add timeout for idle connections

DESCRIPTION: Committing approved fix

BUG#: 5949
TITLE: Content-Type header is not validated

BUG#: 6288
TITLE: Unnecessary null-termination of Buffers
DESCRIPTION: Remove leftover appending of Buffer null-terminator characters.

BUG#: 6275
TITLE: Fix DTDVersion, ProtocolVersion and CIMVersion tests

DESCRIPTION: Moves fix from 2.7 back to 2.6

BUG#: 6022
TITLE:  Modify version tests for spec compliance

DESCRIPTION:  Consolidate DTDVersion protocolVersion CimVersion tests and
test only major version number per spec.

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#: 5691
TITLE: Parameter systemId should be used in the System:syslog() of Logger.cp

DESCRIPTION: Updated System::syslog() in Logger.cpp to use parameter systemId instead of System::CIMSERVER. Also, updated the modules which does not pass proper identifier.

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.

BUG#: 4159
TITLE: ClientPerfDataStore in CIMClientRep.cpp not thread safe

DESCRIPTION:

Changes that assign one ClientPerfDataStore object per CIMClient object.

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#: 4055
TITLE: Change to allow all "1.x" protocol versions.

DESCRIPTION: Changed the code that checks the protocol version of the server to allow all "1.x" versions.  Took the code from the server code that validated the same value from the client.

BUG#: 4054
TITLE: Request message leaked in client authenticator
DESCRIPTION: Use an AutoPtr to hold the request message while it is owned by the ClientAuthenticator.

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

DESCRIPTION: Added changes to reconnect Client with Server when  "Connection: Close" header
is detected during an Authentication Challenge.

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#: 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: check in of rest of code

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.

PEP#: 128

TITLE:Client Statisitcal gathering

DESCRIPTION: function is not done but most of underlying stat. gathering is being checked in now

BUG#: 2297
TITLE: 'Types' changes
USER: David Dillard
MAIL: david.dillard@veritas.com



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#: 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#: 1703
TITLE: [HP-RK] Incorrect XML parsing of empty *METHODRESPONSE elements

DESCRIPTION: Fixed the CIMOperationRequestDecoder and CIMExportRequestDecoder to handle empty *METHODRESPONSE tags.

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

PEP#: 135
TITLE: OperationContext Enhancements.

DESCRIPTION:
Please refer to PEP135 for details.

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.

HP-Nag: Bugzilla:1111 made changes to CIMClient inteface to retain the user information on a TimeoutException like in any other Exception situation.

HP-RK Bugzilla 1023 (CIMClientHTTPErrorException should include reason phrase).

HP-RK Bugzilla 1023 (CIMClientHTTPErrorException should include reason phrase).

HP-RK Bugzilla 1024 (Use CIMClientResponseException for response decoding errors).

HP-RK Bugzilla 1014 (Client application cannot distinguish authentication failure exception).

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

HP-RK Bugzilla 423 (references and referencesNames times out).

IBM:HR:BUG 485

reinsert the header missing code but with conditional compile flag set false per bug 572

fix bug 572, processing of content-type header.  We chose to make the header optional but if it exists check the type and subtype against the Operations over HTTP specification

IBM:HR:BUG 485

HP-RK Bugzilla 508 (Empty string should not be allowed for CIMNamespaceName).

IBM:HR:BUG459

Add client display per PEP 90 to branch

another merge with HEAD

merge with 2.3 localized code

IBM:HR,pep58,R2.3:globalization changes for CIMClient

IBM-DR, PEP58, R2.3: I18N support

add client trace per PEP 90 but with different env var definition

  - Merged CLIENT_TRACE code. This is inside #ifdef PEGASUS_DEBUG wrapper
    and tests for PEGASUS_CLIENT_TRACE in the environment before outputting
    trace data.

g10n changes merged

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

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

HP-RK Bugzilla 135 (Client Response decoder does not handle EMPTY tag).

SF-HP PEP 31: Updated Exception message for Authentication Failure.

HP-RK Phase 1 of fix for Bugzilla 207 (CIMObject Header Not URI-Safe).

add print on MissingCIMOperation Exception

Add console display for some HTTP message errors

Add client trace to std out and bugs from SNIA testing

HP-SF: Fixed Exception message for Authentication Failure. Modified CannotConnectException to accept a message string instead of appending an existing string to the message passed.

HP-[CAKG] API Review: CIMName modifications

HP-RK API Review: Rename KeyBinding to CIMKeyBinding.  Remove definition of KeyBindingArray.

HP-RK API Review: Rename CIMClientHTTPError class to CIMClientHTTPErrorException.

HP-RK API Review: Move CompareNoCase in String.h to System::strcasecmp.

HP-RK API Review: Replace EqualNoCase function with CompareNoCase function in String module.

Changes to compile on OS/400

HP-RK API Review: Remove definitions of CIMClientException and CIMClientCIMException.  Replace with use of Exception and CIMException.

HP-RK API Review: Restructured exceptions to move internal exceptions to an InternalExceptions module.  Combined definitions of overlapping CIMClientExceptions and Exceptions.  Removed unused exception definitions.  This is just the first step in cleaning up the exceptions from a developer API perspective.

HP-[CAKG] Replace CIMType class with an enum CIMType

HP-RK Update license header.

HP-[CAKG] Remove CIMObjectWithPath class; use CIMObject

HP-[CAKG] Remove CIMNamedInstance class; use CIMInstance

HP-WO Added Unauthorized exception.

HP-RK Use CIMObjectPath class instead of CIMReference.

HP-RK Change ClientExceptionMessage to hold a pointer to an exception instead of the exception itself, so that subclass information will not get sliced off during assignment.

HP-RK Improve error handling of bad HTTP response messages.

HP-RK Remove use of String::matchNoCase in HTTPMessage::lookupHeader.  Instead of doing a general pattern match, HTTPMessage::lookupHeader now has specific modes for allowing the HTTP namespace prefix ('dd-') on the header tag or not.  This more closely matches the use model.

HP-RK Defined an exception hierarchy for CIMClient API users.  All exceptions thrown by CIMClient are now subclasses of CIMClientException.  A new message type, ClientExceptionMessage has been defined as well.  This message is used to communicate client exceptions from the CIM operation response decoder to the CIMClient.  The CIM clients that are linked into the Pegasus build have been updated to catch CIMClientException instead of CIMException.

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 Define an HTTP error message to be returned to the CIMClient when an HTTP error response is returned.  Add a method to parse the status line of an HTTP response.

HP-RK Make IRETURNVALUE tag optional on the client side, and don't include that element on the server side when no return value is given.

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 Parse xml tag in CIM requests/responses.

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

HP-RK Implement HTTP header consistency validation for CIMProtocolVersion headers.

HP-Nag: Fixed a bug in the ClientAuthenticator code.

HP-RK Clean up decoding of property values in SetProperty requests and GetProperty responses.

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.

HP-RK Updated the CIMOperationResponseDecoder to accept InvokeMethod responses with the return value and the parameter values in any order.

undid aysynchronous message queue due to problems running TestClient

Merged dev branch into main trunk.

HP-RK Throw exceptions in _handleMethodResponse instead of ignoring errors.

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

HP-RK Fixed a memory error caused by the Decoders appending a null character to an HTTP message after getting a pointer into an array returned by HTTPMessage::parse() and before using it.  Since Array::append() can move the array contents, the pointer may become invalid.

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-NB Implemented Authentication code on the server to challenge requests, authenticate the challenge responses, and verify the subsequent authorized requests. Implemented authentication mechanism in the Client API to send responses to the challenges from the server. Current implementation supports only 'local' authentication mechanism. Authentication can be enabled by setting the config property 'requireAuthentication' to true.

Fixed compiler warnings on Linux/GNU C++.

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

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

HP-[Nitin Upasani] Decoders are changed to parse Xml completely before sending request and response.

HP-[Nitin Upasani] Implementation of InvokeMethod.

Finalized queue-based client rework.

Finished implementation of CIMOperationResponseDecoder class as part of
reworking the client into message queues.

More implementation

New files

file CIMOperationResponseDecoder.cpp was initially added on branch dev.