vArmour's RESTful API response should handle more error code besides HTTP 200 OK

Bug #1222973 reported by Gary Duan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Medium
Gary Duan

Bug Description

When handling rest API responses, vArmour's rest API interface only handle HTTP 200 OK. It cannot distinguish 400, 401 and 404 etc.

Properly handle these response code will improve the robustness and performance of the driver.

Revision history for this message
Gary Duan (gduan) wrote :

This bug request is also based on Salvatore's code review on vArmour's FWaaS driver implementation. We didn't have time to address it at the time.

Changed in neutron:
status: New → In Progress
assignee: nobody → Gary Duan (gduan)
tags: added: fwaas varmour
Changed in neutron:
importance: Undecided → Medium
milestone: none → havana-rc1
Changed in neutron:
milestone: havana-rc1 → none
Revision history for this message
Eugene Nikanorov (enikanorov) wrote :

Is this still a relevant bug?

Revision history for this message
Gary Duan (gduan) wrote : Re: [Bug 1222973] Re: vArmour's RESTful API response should handle more error code besides HTTP 200 OK

Hi, Eugene,

It is not really relevant now. We have refactored our plugin and driver and
have been planning to upstream it.

Thanks,
Gary

On Sun, Nov 23, 2014 at 10:19 PM, Eugene Nikanorov <email address hidden>
wrote:

> Is this still a relevant bug?
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1222973
>
> Title:
> vArmour's RESTful API response should handle more error code besides
> HTTP 200 OK
>
> Status in OpenStack Neutron (virtual network service):
> In Progress
>
> Bug description:
> When handling rest API responses, vArmour's rest API interface only
> handle HTTP 200 OK. It cannot distinguish 400, 401 and 404 etc.
>
> Properly handle these response code will improve the robustness and
> performance of the driver.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/neutron/+bug/1222973/+subscriptions
>

Revision history for this message
Eugene Nikanorov (enikanorov) wrote :

Closing per previous comment

Changed in neutron:
status: In Progress → Invalid
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers