Subscription manager should respond with complete PNStatus

[gvaibhav1734]

PNStatus not populated completely

go/subscription_manager.go

Line 336 in a2c70fc

res, _, err := executeRequest(opts)

Why are we ignoring the PNStatus returned by executeRequest here?

executeRequest has full context of the API response and returns a fully populated PNStatus which is ignored and it is then being recreated minimally with only PNStatus.Category information.

Solution

We should either be passing the returned response from executeRequest or construct the PNStatus fully.