@villedemontreal/http-request 7.4.6 → 7.4.8
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +10 -5
- package/dist/config/configs.d.ts +16 -0
- package/dist/config/configs.js +15 -0
- package/dist/config/configs.js.map +1 -0
- package/dist/scripts/index.d.ts +2 -0
- package/dist/scripts/index.js +14 -0
- package/dist/scripts/index.js.map +1 -0
- package/dist/scripts/lint.d.ts +6 -0
- package/dist/scripts/lint.js +31 -0
- package/dist/scripts/lint.js.map +1 -0
- package/dist/scripts/lintFix.d.ts +6 -0
- package/dist/scripts/lintFix.js +40 -0
- package/dist/scripts/lintFix.js.map +1 -0
- package/dist/src/config/configs.d.ts +1 -1
- package/dist/src/config/configs.js.map +1 -1
- package/dist/src/config/constants.d.ts +1 -1
- package/dist/src/config/constants.js +2 -2
- package/dist/src/config/constants.js.map +1 -1
- package/dist/src/config/init.js.map +1 -1
- package/dist/src/httpUtils.d.ts +1 -1
- package/dist/src/httpUtils.js +84 -73
- package/dist/src/httpUtils.js.map +1 -1
- package/dist/src/httpUtils.test.js +182 -169
- package/dist/src/httpUtils.test.js.map +1 -1
- package/dist/src/utils/logger.js.map +1 -1
- package/package.json +10 -9
- package/src/config/configs.ts +6 -4
- package/src/config/constants.ts +3 -3
- package/src/config/init.ts +2 -2
- package/src/httpUtils.test.ts +99 -53
- package/src/httpUtils.ts +15 -13
- package/src/utils/logger.ts +8 -1
package/src/httpUtils.ts
CHANGED
|
@@ -46,7 +46,7 @@ export class HttpUtils {
|
|
|
46
46
|
*/
|
|
47
47
|
public urlJoin(...args: string[]) {
|
|
48
48
|
return _.map(args, this.removeSlashes)
|
|
49
|
-
.filter(x => !!x)
|
|
49
|
+
.filter((x) => !!x)
|
|
50
50
|
.join('/');
|
|
51
51
|
}
|
|
52
52
|
|
|
@@ -83,12 +83,14 @@ export class HttpUtils {
|
|
|
83
83
|
if ('status' in request) {
|
|
84
84
|
throw new Error(
|
|
85
85
|
`The request object must be of type SuperAgentRequest. Make sure this object has NOT already been awaited ` +
|
|
86
|
-
`prior to being passed here
|
|
86
|
+
`prior to being passed here!`,
|
|
87
87
|
);
|
|
88
88
|
}
|
|
89
89
|
|
|
90
90
|
if (!request.url || request.url.indexOf('://') < 0) {
|
|
91
|
-
throw new Error(
|
|
91
|
+
throw new Error(
|
|
92
|
+
`The URL in your request MUST have a protocol and a hostname. Received: ${request.url}`,
|
|
93
|
+
);
|
|
92
94
|
}
|
|
93
95
|
|
|
94
96
|
if (utils.isBlank(request.get(httpHeaderFieldsTyped.X_CORRELATION_ID))) {
|
|
@@ -111,14 +113,14 @@ export class HttpUtils {
|
|
|
111
113
|
deadline:
|
|
112
114
|
request[timeoutRequestVarName] !== undefined
|
|
113
115
|
? request[timeoutRequestVarName]
|
|
114
|
-
: constants.request.timeoutsDefault.deadline
|
|
116
|
+
: constants.request.timeoutsDefault.deadline,
|
|
115
117
|
});
|
|
116
118
|
|
|
117
119
|
logger.debug({
|
|
118
120
|
sendingCorrelationIdHeader: request.get(httpHeaderFieldsTyped.X_CORRELATION_ID) || null,
|
|
119
121
|
url: request.url,
|
|
120
122
|
method: request.method,
|
|
121
|
-
msg: `Http Client - Start request to ${request.method} ${request.url}
|
|
123
|
+
msg: `Http Client - Start request to ${request.method} ${request.url}`,
|
|
122
124
|
});
|
|
123
125
|
|
|
124
126
|
let result;
|
|
@@ -146,12 +148,12 @@ export class HttpUtils {
|
|
|
146
148
|
url: request.url,
|
|
147
149
|
method: request.method,
|
|
148
150
|
timeTaken: timer.toString(),
|
|
149
|
-
msg: `Http Client - End request ERROR request to ${request.method} ${request.url}
|
|
151
|
+
msg: `Http Client - End request ERROR request to ${request.method} ${request.url}`,
|
|
150
152
|
});
|
|
151
153
|
|
|
152
154
|
throw {
|
|
153
155
|
msg: `An error occured while making the HTTP request to ${request.method} ${request.url}`,
|
|
154
|
-
originalError: err
|
|
156
|
+
originalError: err,
|
|
155
157
|
};
|
|
156
158
|
}
|
|
157
159
|
}
|
|
@@ -161,7 +163,7 @@ export class HttpUtils {
|
|
|
161
163
|
method: request.method,
|
|
162
164
|
statusCode: result.status,
|
|
163
165
|
timeTaken: timer.toString(),
|
|
164
|
-
msg: `Http Client - End request to ${request.method} ${request.url}
|
|
166
|
+
msg: `Http Client - End request to ${request.method} ${request.url}`,
|
|
165
167
|
});
|
|
166
168
|
|
|
167
169
|
return result;
|
|
@@ -263,7 +265,7 @@ export class HttpUtils {
|
|
|
263
265
|
public getQueryParamOneAsDate = (
|
|
264
266
|
req: Request,
|
|
265
267
|
key: string,
|
|
266
|
-
errorHandler?: (errMsg: string, value?: string) => any
|
|
268
|
+
errorHandler?: (errMsg: string, value?: string) => any,
|
|
267
269
|
): Date => {
|
|
268
270
|
const dateStr = this.getQueryParamOne(req, key);
|
|
269
271
|
let date: Date;
|
|
@@ -301,7 +303,7 @@ export class HttpUtils {
|
|
|
301
303
|
public getQueryParamOneAsNumber = (
|
|
302
304
|
req: Request,
|
|
303
305
|
key: string,
|
|
304
|
-
errorHandler?: (errMsg: string, value?: string) => any
|
|
306
|
+
errorHandler?: (errMsg: string, value?: string) => any,
|
|
305
307
|
): number => {
|
|
306
308
|
const numberStr = this.getQueryParamOne(req, key);
|
|
307
309
|
let val: number;
|
|
@@ -332,7 +334,7 @@ export class HttpUtils {
|
|
|
332
334
|
public getQueryParamOneAsBoolean = (
|
|
333
335
|
req: Request,
|
|
334
336
|
key: string,
|
|
335
|
-
errorHandler?: (errMsg: string, value?: string) => any
|
|
337
|
+
errorHandler?: (errMsg: string, value?: string) => any,
|
|
336
338
|
): boolean => {
|
|
337
339
|
const boolStr = this.getQueryParamOne(req, key);
|
|
338
340
|
if (utils.isBlank(boolStr)) {
|
|
@@ -394,7 +396,7 @@ export class HttpUtils {
|
|
|
394
396
|
|
|
395
397
|
const orderBy: IOrderBy = {
|
|
396
398
|
key,
|
|
397
|
-
direction
|
|
399
|
+
direction,
|
|
398
400
|
};
|
|
399
401
|
orderBys.push(orderBy);
|
|
400
402
|
}
|
|
@@ -402,4 +404,4 @@ export class HttpUtils {
|
|
|
402
404
|
return orderBys;
|
|
403
405
|
};
|
|
404
406
|
}
|
|
405
|
-
export
|
|
407
|
+
export const httpUtils: HttpUtils = new HttpUtils();
|
package/src/utils/logger.ts
CHANGED
|
@@ -1,4 +1,11 @@
|
|
|
1
|
-
import {
|
|
1
|
+
import {
|
|
2
|
+
ILogger,
|
|
3
|
+
initLogger,
|
|
4
|
+
LazyLogger,
|
|
5
|
+
Logger,
|
|
6
|
+
LoggerConfigs,
|
|
7
|
+
LogLevel,
|
|
8
|
+
} from '@villedemontreal/logger';
|
|
2
9
|
import { configs } from '../config/configs';
|
|
3
10
|
|
|
4
11
|
let testingLoggerLibInitialised = false;
|