post body validation not working correctly with pecan

[Doug Hellmann]

In ceilometer we have had a problem with WSME doing the type conversion for us for POST bodies.

http://git.openstack.org/cgit/openstack/ceilometer/tree/ceilometer/api/controllers/v2.py#n576

It's not clear if this is a bug in WSME, something specific to the pecan adapter, or something about the way we're using the decorators for that function.

[Feilong Wang (flwang)]

Doug, why we need a new bug to track this issue? Since I have opened one: https://bugs.launchpad.net/wsme/+bug/1218760

[Doug Hellmann (doug-hellmann)]

Fei Long, I think these are two separate issues. In this ticket I am trying to describe a case where WSME is not properly deserializing the incoming body of a POST. There's no error, the body is just give to us as a dict() instead of the complex type.

[Feilong Wang (flwang)]

Got it, Doug. I will continue keeping an eye on this. Thanks.

[Julien Danjou (jdanjou)]

Doug, it'd be great if you had a test case to reproduce this.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/50667
Committed: http://github.com/stackforge/wsme/commit/b67405d486a0d09abde6423818734bbb5a978bc4
Submitter: Jenkins
Branch: master

commit b67405d486a0d09abde6423818734bbb5a978bc4
Author: Julien Danjou <email address hidden>
Date: Wed Oct 9 17:07:40 2013 +0200

    Validate body when using Pecan

    This makes sure that when the body is decoded we run the validation
    process on it.

    Fixes-Bug: #1220678

    Change-Id: I765a6ce8c8097ef0e0a734804242e9fcee4ffbcf

[Ildiko Vancsa (ildiko-vancsa)]

Hi,

Is this still a valid bug for Ceilometer?

Best Regards,
Ildiko

[Steve Lewis (steve-lewis)]

The fix for WSME was released 0.5b6

[Steve Lewis (steve-lewis)]

I verified that this dependency was updated in Icehouse before the b1 milestone