Fix documentation, comment and condition for getEndpointAuthorization function (#790)

* fix doc, comment and condition

* bumped version

* bump version

* fix documentation

Co-authored-by: Anna Opareva <[email protected]>
Co-authored-by: Simon Alling <[email protected]>

Author: 3 people

node/docs/azure-pipelines-task-lib.md

@@ -616,7 +616,7 @@ scheme | string | auth scheme such as OAuth or username/password etc...
 
 ### task.getEndpointAuthorization <a href="#index">(^)</a>
 Gets the authorization details for a service endpoint
-If the authorization was not set and is not optional, it will throw.
+If the authorization was not set and is not optional, it will set the task result to Failed.
 
 @returns   string
 ```javascript

node/package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "azure-pipelines-task-lib",
-  "version": "3.1.8",
+  "version": "3.1.9",
   "description": "Azure Pipelines Task SDK",
   "main": "./task.js",
   "typings": "./task.d.ts",

node/package.json

@@ -407,7 +407,7 @@ export interface EndpointAuthorization {
 
 /**
  * Gets the authorization details for a service endpoint
- * If the authorization was not set and is not optional, it will throw.
+ * If the authorization was not set and is not optional, it will set the task result to Failed.
  *
  * @param     id        name of the service endpoint
  * @param     optional  whether the url is optional
@@ -420,7 +420,7 @@ export function getEndpointAuthorization(id: string, optional: boolean): Endpoin
         setResult(TaskResult.Failed, loc('LIB_EndpointAuthNotExist', id));
     }
 
-    debug(id + ' exists ' + (aval !== null));
+    debug(id + ' exists ' + (!!aval));
 
     var auth: EndpointAuthorization | undefined;
     try {