Simple pub sub integration #113
Conversation
|
|
|
Coverity detected 2 issues; a quality concern. |
| }; | ||
| requestMap.params.intent.data = data; | ||
| } | ||
|
|
||
| // Add the PubSub URL if required | ||
| if (getEnvVariable(CONSTANTS.PUB_SUB_URL, false)) { | ||
| data.query.params[CONSTANTS.PUB_SUB_URL] = getEnvVariable(CONSTANTS.PUB_SUB_URL); |
There was a problem hiding this comment.
Medium impact. Quality issue.
Null pointer dereferences. Bad use of null-like value.
Performing an operation that would be invalid on a null or undefined value.
If the value is null or undefined, this code will cause an exception such as a TypeError.
| } | ||
|
|
||
| // Stringify the query (The intent requires it be a string) | ||
| data.query = JSON.stringify(data.query); |
There was a problem hiding this comment.
Medium impact. Quality issue.
Null pointer dereferences. Bad use of null-like value.
Performing an operation that would be invalid on a null or undefined value.
If the value is null or undefined, this code will cause an exception such as a TypeError.
|
Issue with linting or unit tests detected. For more information on our linting policies, please see our Linting-Guide. |
|
Issue with linting or unit tests detected. For more information on our linting policies, please see our Linting-Guide. |
No description provided.