On Thu, Jun 4, 2015 at 4:43 AM Brant Knudson <[email protected]> wrote:

>
>
> On Tue, Jun 2, 2015 at 4:32 AM, Chenhong Liu <[email protected]>
> wrote:
>
>> There is keystone/exception.py which contains Exceptions defined and used
>> inside keystone provide 4xx and 5xx status code. And we can use it like:
>>     exception.Forbidden.code, exception.forbiddent.title
>>     exception.NotFound.code, exception.NotFound.title
>>
>> This makes the code looks pretty and avoid error prone. But I can't find
>> definition for other status code, like 200, 201, 204, 302, and so on. The
>> code in keystone, especially the unit test cases,  just write these status
>> code and title explicitly.
>>
>> How about add those definitions?
>>
>>
> Looks like Python 3.5 added constants in http.HTTPStatus[1]. So someday
> we'll be able to use those instead.
>
> [1] https://docs.python.org/3.5/library/http.html#http.HTTPStatus
>
> Sincerely, Brant
>
>
This will be a good choice.


>
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: [email protected]?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: [email protected]?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Reply via email to