Skip to content

Commit

Permalink
Move injected types into internal types
Browse files Browse the repository at this point in the history
  • Loading branch information
simonhoibm committed Jun 3, 2017
1 parent fd00595 commit b9c2411
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 9 deletions.
9 changes: 7 additions & 2 deletions packages/core/src/http-handler.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,14 @@ import {ServerRequest, ServerResponse} from 'http';
import {getApiSpec} from './router/metadata';
import * as HttpErrors from 'http-errors';

import {Sequence, FindRoute, InvokeMethod} from './sequence';
import {Sequence} from './sequence';
import {RoutingTable, parseRequestUrl} from './router/routing-table';
import {ParsedRequest, OperationArgs} from './internal-types';
import {
FindRoute,
InvokeMethod,
ParsedRequest,
OperationArgs,
} from './internal-types';

const debug = require('debug')('loopback:core:http-handler');

Expand Down
5 changes: 5 additions & 0 deletions packages/core/src/internal-types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
// License text available at https://opensource.org/licenses/MIT

import {ServerRequest} from 'http';
import {ResolvedRoute} from './router/routing-table';

export interface ParsedRequest extends ServerRequest {
// see http://expressjs.com/en/4x/api.html#req.path
Expand All @@ -16,6 +17,10 @@ export interface ParsedRequest extends ServerRequest {
method: string;
}

export type FindRoute = (request: ParsedRequest) => ResolvedRoute<string>;
export type InvokeMethod = (controller: string, method: string, args: OperationArgs) => Promise<OperationRetval>;
export type LogError = (err: Error, statusCode: number, request: ServerRequest) => void;

// tslint:disable:no-any
export type PathParameterValues = {[key: string]: any};
export type OperationArgs = any[];
Expand Down
10 changes: 3 additions & 7 deletions packages/core/src/sequence.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,10 @@ import {Application} from './application';
const debug = require('debug')('loopback:core:sequence');
import {ServerRequest, ServerResponse} from 'http';
import {inject} from '@loopback/context';
import {ResolvedRoute} from './router/routing-table';
import {
FindRoute,
InvokeMethod,
LogError,
OperationArgs,
OperationRetval,
ParsedRequest,
Expand All @@ -17,12 +19,6 @@ import {parseOperationArgs} from './parser';
import {writeResultToResponse} from './writer';
import {HttpError} from 'http-errors';

export type FindRoute = (request: ParsedRequest) => ResolvedRoute<string>;
export type InvokeMethod =
(controller: string, method: string, args: OperationArgs) => Promise<OperationRetval>;
export type LogError =
(err: Error, statusCode: number, request: ServerRequest) => void;

export class Sequence {
constructor(
@inject('findRoute')
Expand Down

0 comments on commit b9c2411

Please sign in to comment.