RestController should return HTTP 405

[Ryan Petrello]

RestController has a variety of branches which return HTTP 404 (and some that correctly return 405). Audit the code and properly return HTTP 405 in situations where a specified method is not supported (instead of an HTTP 404). Additionally, update RestController to properly return an "Allow:" header so that API consumers know which request methods are allowed.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/182273
Committed: https://git.openstack.org/cgit/openstack/designate/commit/?id=35b6d607e130a8c2b5db3553e52cf71d93bded2c
Submitter: Jenkins
Branch: master

commit 35b6d607e130a8c2b5db3553e52cf71d93bded2c
Author: Ryan Petrello <email address hidden>
Date: Tue May 12 08:50:50 2015 -0400

    Remove unnecessary RestController usage

    TasksControllers is just an intermediate path segment, and doesn't need to be
    a RestController. Additionally, this resolves a failing test associated with
    a RestController bug fix in pecan [1].

    [1] https://review.openstack.org/#/c/181979/
    Closes-Bug: #1334690

    Change-Id: I90b8f499f23762cbc6333be764439104189238b8

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/181979
Committed: https://git.openstack.org/cgit/stackforge/pecan/commit/?id=3c634de56666cfadf19b813dd9de4e7e66b6ac65
Submitter: Jenkins
Branch: master

commit 3c634de56666cfadf19b813dd9de4e7e66b6ac65
Author: Ryan Petrello <email address hidden>
Date: Mon May 11 09:48:53 2015 -0700

    Properly raise HTTP 405 (and specify Allow headers) for RestController

    Change-Id: Id790efc75c8207eb61d74e9b2242b310ccd62ab1
    Depends-On: Ieffa3fddc3c8d3152742455ca46d69bcc7208d69
    Closes-bug: #1334690
    Closes-bug: #1450109

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/185684

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/185684
Committed: https://git.openstack.org/cgit/openstack/designate/commit/?id=c7b7975d877fc09462f4f761ed2527a0db1e062a
Submitter: Jenkins
Branch: stable/kilo

commit c7b7975d877fc09462f4f761ed2527a0db1e062a
Author: Ryan Petrello <email address hidden>
Date: Tue May 12 08:50:50 2015 -0400

    Remove unnecessary RestController usage

    TasksControllers is just an intermediate path segment, and doesn't need to be
    a RestController. Additionally, this resolves a failing test associated with
    a RestController bug fix in pecan [1].

    [1] https://review.openstack.org/#/c/181979/
    Closes-Bug: #1334690

    Change-Id: I90b8f499f23762cbc6333be764439104189238b8
    (cherry picked from commit 35b6d607e130a8c2b5db3553e52cf71d93bded2c)

[gordon chung (chungg)]

addressed by:

https://review.openstack.org/#/c/182306/ and https://github.com/openstack/ceilometer/commit/4a0aceecaa2a8e516df33e82e4beae17a68b9f49