No Error Message for missing Restrictions Rules property

Bug #1406380 reported by William Brothers
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Poppy
Fix Released
Medium
Amit Gandhi

Bug Description

Post a request
a valid name value
a domain entry with a domain property set to "invalid"
an origin entry with the origin property set to a "not an origin" and port property set to 80 and the ssl property set to true
an origin entry with a rules entry with the name property set to "name." and request_url set to "www.somewebsite.com"
a caching entry with a name property set to "caching name", the ttl property set to 0, and rules name set to "rules name" and request_url set to "www.somewhere.com"
a restrictions entry that has no name or rules list
{
  "name": "Bill's New CDN Service",
  "domains": [{
                "domain": "invalid"
              }],
  "origins": [{
                "origin": "invalid",
    "port": 80,
    "ssl": true,
    "rules":[{"name": "name.",
              "request_url": "www.somewebsite.com"}]
              }],
  "caching":[{ "name": "caching name",
               "ttl": 0,
        "rules": [{"name": "rules name", "request_url":"www.somewhere.com"}]
             }],
  "restrictions":[{}]
}

Expected Result:
400 status and error message stating that restrictions name property is required and the restrictions rules property is required.

Actual Result:
400 status with the following message:
{
"message": "{\"errors\": [{\"message\": \"['restrictions'][0]['name']-'name' is a required property\"}, {\"message\": \"['flavor_id']-'flavor_id' is a required property\"}]}"
}

Changed in poppy:
status: New → Confirmed
milestone: none → kilo-2
Changed in poppy:
assignee: nobody → Amit Gandhi (amit-gandhi)
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to poppy (master)

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

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

Reviewed: https://review.openstack.org/145341
Committed: https://git.openstack.org/cgit/stackforge/poppy/commit/?id=863920b5c63ca165babb846c8b3fe0e2b6f25145
Submitter: Jenkins
Branch: master

commit 863920b5c63ca165babb846c8b3fe0e2b6f25145
Author: amitgandhinz <email address hidden>
Date: Tue Jan 6 16:07:28 2015 -0500

    Add rules for restrictions

    Change-Id: I0403276ff2aa62c544816e8d5f13d0b2c3dcebc0
    Closes-Bug: 1406380
    Closes-Bug: 1406384
    Closes-Bug: 1406348

Changed in poppy:
status: In Progress → Fix Committed
Changed in poppy:
status: Fix Committed → Fix Released
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.