@loopback/rest 1.24.0 → 1.26.1

package/src/router/handler-route.ts

@@ -7,7 +7,7 @@ import {Context, invokeMethodWithInterceptors} from '@loopback/context';
 import {OperationObject} from '@loopback/openapi-v3';
 import {RestBindings} from '../keys';
 import {OperationArgs, OperationRetval} from '../types';
-import {BaseRoute} from './base-route';
+import {BaseRoute, RouteSource} from './base-route';
 
 export class Route extends BaseRoute {
   constructor(
@@ -33,6 +33,12 @@ export class Route extends BaseRoute {
   ): Promise<OperationRetval> {
     // Use `invokeMethodWithInterceptors` to invoke the handler function so
     // that global interceptors are applied
-    return invokeMethodWithInterceptors(requestContext, this, '_handler', args);
+    return invokeMethodWithInterceptors(
+      requestContext,
+      this,
+      '_handler',
+      args,
+      {source: new RouteSource(this)},
+    );
   }
 }

package/src/router/router-base.ts

@@ -109,7 +109,7 @@ export abstract class BaseRouter implements RestRouter {
  */
 function normalizeVerb(verb: string) {
   // Use lower case, default to `get`
-  return (verb && verb.toLowerCase()) || 'get';
+  return verb?.toLowerCase() || 'get';
 }
 
 /**

package/src/router/routing-table.ts

@@ -9,9 +9,9 @@ import {
   ParameterObject,
   PathObject,
 } from '@loopback/openapi-v3';
-import * as assert from 'assert';
-import * as debugFactory from 'debug';
-import * as HttpErrors from 'http-errors';
+import assert from 'assert';
+import debugFactory from 'debug';
+import HttpErrors from 'http-errors';
 import {inspect} from 'util';
 import {Request} from '../types';
 import {
@@ -57,7 +57,7 @@ export class RoutingTable {
 
     debug('Registering Controller with API %s', inspect(spec, {depth: null}));
 
-    const basePath = spec.basePath || '/';
+    const basePath = spec.basePath ?? '/';
     for (const p in spec.paths) {
       for (const verb in spec.paths[p]) {
         const opSpec: OperationObject = spec.paths[p][verb];
@@ -154,6 +154,6 @@ export class RoutingTable {
 
 function describeOperationParameters(opSpec: OperationObject) {
   return ((opSpec.parameters as ParameterObject[]) || [])
-    .map(p => (p && p.name) || '')
+    .map(p => p?.name || '')
     .join(', ');
 }

package/src/router/trie-router.ts

@@ -42,7 +42,7 @@ export class TrieRouter extends BaseRouter {
       const route = found.node.value!;
       if (route) {
         debug('Route found: %s', inspect(route, {depth: 5}));
-        return createResolvedRoute(route, found.params || {});
+        return createResolvedRoute(route, found.params ?? {});
       }
     }
     return undefined;

package/src/types.ts

@@ -5,7 +5,7 @@
 
 import {Binding, BoundValue} from '@loopback/context';
 import {ReferenceObject, SchemaObject} from '@loopback/openapi-v3';
-import * as ajv from 'ajv';
+import ajv from 'ajv';
 import {
   Options,
   OptionsJson,

package/src/validation/request-body.validator.ts

@@ -10,10 +10,10 @@ import {
   SchemaObject,
   SchemasObject,
 } from '@loopback/openapi-v3';
-import * as AJV from 'ajv';
-import * as debugModule from 'debug';
-import * as _ from 'lodash';
-import * as util from 'util';
+import AJV from 'ajv';
+import debugModule from 'debug';
+import _ from 'lodash';
+import util from 'util';
 import {HttpErrors, RequestBody, RestHttpErrors} from '..';
 import {RequestBodyValidationOptions, SchemaValidatorCache} from '../types';
 
@@ -38,7 +38,7 @@ export function validateRequestBody(
   globalSchemas: SchemasObject = {},
   options: RequestBodyValidationOptions = {},
 ) {
-  const required = requestBodySpec && requestBodySpec.required;
+  const required = requestBodySpec?.required;
 
   if (required && body.value == null) {
     const err = Object.assign(
@@ -125,7 +125,7 @@ function validateValueAgainstSchema(
 ) {
   let validate: AJV.ValidateFunction | undefined;
 
-  const cache = options.compiledSchemaCache || DEFAULT_COMPILED_SCHEMA_CACHE;
+  const cache = options.compiledSchemaCache ?? DEFAULT_COMPILED_SCHEMA_CACHE;
   const key = getKeyForOptions(options);
 
   let validatorMap: Map<string, AJV.ValidateFunction> | undefined;
@@ -136,7 +136,7 @@ function validateValueAgainstSchema(
 
   if (!validate) {
     validate = createValidator(schema, globalSchemas, options);
-    validatorMap = validatorMap || new Map();
+    validatorMap = validatorMap ?? new Map();
     validatorMap.set(key, validate);
     cache.set(schema, validatorMap);
   }

package/src/writer.ts

@@ -31,7 +31,7 @@ export function writeResultToResponse(
   }
 
   const isStream =
-    result instanceof Readable || typeof (result && result.pipe) === 'function';
+    result instanceof Readable || typeof result?.pipe === 'function';
 
   if (isStream) {
     response.setHeader('Content-Type', 'application/octet-stream');