Fix: Web Socket Invocation Error with Tracing Enabled

[BLasan]

Purpose

$subject

Issue

• Can't invoke the WebSocket API after enabling the open tracing. api-manager#1550

[codecov]

Codecov Report

Patch coverage: 15.55% and project coverage change: -0.60 ⚠️

Comparison is base (5c49840) 38.66% compared to head (d2dc4b2) 38.06%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master   #11936      +/-   ##
==========================================
- Coverage   38.66%   38.06%   -0.60%     
==========================================
  Files        1778     1778              
  Lines      127466   127475       +9     
  Branches    18213    18220       +7     
==========================================
- Hits        49288    48527     -761     
- Misses      71925    72751     +826     
+ Partials     6253     6197      -56     

Flag	Coverage Δ
integration_tests	30.49% <15.55%> (-0.72%)	⬇️
unit_tests	20.57% <0.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...y/handlers/common/APIMgtLatencySynapseHandler.java	10.86% <0.00%> (-0.90%)	⬇️
...wso2/carbon/apimgt/gateway/utils/GatewayUtils.java	41.93% <0.00%> (ø)
...2/carbon/apimgt/keymgt/SubscriptionDataHolder.java	85.71% <77.77%> (-2.75%)	⬇️

... and 71 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[Arshardh]

Did we check if there are any possible null pointers that can occur as a result of sending null.

[BLasan]

When analyzing the master branch, line 80 shouldn't get hit. This null check is added in https://github.com/wso2/carbon-apimgt/blob/master/components/apimgt/org.wso2.carbon.apimgt.keymgt/src/main/java/org/wso2/carbon/apimgt/keymgt/handlers/AbstractKeyValidationHandler.java#L219