IStorageServiceClient.UpdateStorageObject not working

Bug #1331802 reported by Wayne Foley
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openstack-sdk-dotnet
Fix Committed
High
Wayne Foley

Bug Description

Updating storage object metadata via the UpdateStorageObject method on the StorageServiceClient class fails with an argument null exception.

RCA: The HttpAbstraction layer is trying to create a request body using the stream that is passed in from the StorageServiceRestClient. The object that is passed in from the user does not (and will not) have it's Content property set, and is defaulted to null.

The HttpAbstraction layer should detected the null stream, and send an empty content body instead of throwing an exception.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-sdk-dotnet (master)

Fix proposed to branch: master
Review: https://review.openstack.org/103659

Changed in openstack-sdk-dotnet:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-sdk-dotnet (master)

Reviewed: https://review.openstack.org/103659
Committed: https://git.openstack.org/cgit/stackforge/openstack-sdk-dotnet/commit/?id=2675cd745b796a26702c3340f5ace0ce01ca91dc
Submitter: Jenkins
Branch: master

commit 2675cd745b796a26702c3340f5ace0ce01ca91dc
Author: Wayne Foley <email address hidden>
Date: Mon Jun 30 13:57:13 2014 -0700

    Fix for bug #1331802.

    Updated the HttpAbstractionClient code to check for a null content buffer when trying to send a POST or PUT request.
    Added a unit test case to check for this bug.

    Closes-Bug: #1331802
    Change-Id: I99d93e77a0a5ca4c3678c825517050ab4e3f6789

Changed in openstack-sdk-dotnet:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.