open-traffic-generator/snappi

snappi 0.7.13 grpc transport is returning errors without status code

biplamal opened this issue · 2 comments

for bad json & invalid values/types snappi grpc transport raising exception with out any status code.

http-transport raised exception:

expected_exception = Exception(
400, {
'errors': (
[
'unmarshal error syntax error (line 3:23):'
' unexpected token "rx"']
)
}
)

grpc-transport raised exception:

expected_exception = Exception(
'Failed to parse port_names field: repeated field '
'port_names must be in [] which is rx..'
)

We are raising exception (with error code, errors) when we will get some error response (not 200) from server. But in this case the exception is coming before send it to channel. So, I am little concern to raise a response code in this case. Because we are getting this from deserialization in grpc pb2 format which is nothing but a wrapper code on top of grpc channel. As well as errors should differ.

Got your point