Skip to content

Commit 7fa5515

Browse files
authored
skip healthcheck (#168)
* skip healthcheck * update biome * biome fixes
1 parent 156d39f commit 7fa5515

File tree

15 files changed

+148
-32
lines changed

15 files changed

+148
-32
lines changed

.github/workflows/test.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,6 @@ jobs:
99
with:
1010
node-version: 21
1111
- run: npm ci
12-
- uses: biomejs/setup-biome@v1
12+
- uses: biomejs/setup-biome@v2
1313
- run: biome ci .
1414
- run: npm test

lib/middleware/body-parser.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
const Dicer = require("dicer");
2-
const querystring = require("querystring");
2+
const querystring = require("node:querystring");
33
const contentTypeParser = require("content-type");
44
const util = require("../utils");
55

lib/middleware/negotiate-content.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ module.exports = (req, res, next) => {
88

99
// am I pretty?
1010
const spaces = req.headers["x-pretty-print"]
11-
? parseInt(req.headers["x-pretty-print"], 10)
11+
? Number.parseInt(req.headers["x-pretty-print"], 10)
1212
: 2;
1313

1414
const xmlOpts = {

lib/routes/bins.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ const express = require("express");
33
const mw = require("../middleware");
44
const { createClient } = require("redis");
55
const routes = require("./bins/");
6-
const URL = require("url").URL;
6+
const URL = require("node:url").URL;
77

88
module.exports = function bins(dsnStr) {
99
// parse redis dsn

lib/routes/bins/create.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
const debug = require("debug")("mockbin");
2-
const util = require("util");
2+
const util = require("node:util");
33
const uuid = require("uuid");
44
const validate = require("har-validator");
55

lib/routes/bins/sample.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
const debug = require("debug")("mockbin");
2-
const util = require("util");
2+
const util = require("node:util");
33

44
module.exports = (client) => (req, res, next) => {
55
client.get(`bin:${req.params.uuid}`, (err, value) => {

lib/routes/bins/update.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
const debug = require("debug")("mockbin");
22
const e = require("express");
33
const validate = require("har-validator");
4-
const path = require("path");
4+
const path = require("node:path");
55

66
module.exports = (client) => (req, res, next) => {
77
const id = req.params.uuid;

lib/routes/delay.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
module.exports = function delay(req, res, next) {
2-
let delay = req.params.ms ? parseInt(req.params.ms, 10) : 200;
2+
let delay = req.params.ms ? Number.parseInt(req.params.ms, 10) : 200;
33

44
if (delay > 60000) {
55
delay = 60000;

lib/routes/redirect.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
1-
const util = require("util");
1+
const util = require("node:util");
22

33
module.exports = function redirect(req, res, next) {
4-
let count = req.params.count ? parseInt(req.params.count, 10) : 1;
5-
const status = parseInt(req.params.status_code, 10) || 302;
4+
let count = req.params.count ? Number.parseInt(req.params.count, 10) : 1;
5+
const status = Number.parseInt(req.params.status_code, 10) || 302;
66
const valid = [300, 301, 302, 303, 307, 308];
77

88
if (count > 100) {

lib/routes/stream.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ module.exports = function stream(req, res, next) {
55
});
66

77
// set default chunks to 10
8-
let chunks = req.params.chunks ? parseInt(req.params.chunks, 10) : 10;
8+
let chunks = req.params.chunks ? Number.parseInt(req.params.chunks, 10) : 10;
99

1010
// max out chunks at 100
1111
if (chunks > 100) {

package-lock.json

+129-15
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@
4343
"clean": "git clean -fdX"
4444
},
4545
"devDependencies": {
46-
"@biomejs/biome": "1.4.1",
46+
"@biomejs/biome": "1.6.4",
4747
"mocha": "^10.2.0",
4848
"should": "^13.2.3"
4949
},

src/index.js

+4-2
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ const debug = require("debug")("mockbin");
44
const express = require("express");
55
const methodOverride = require("method-override");
66
const morgan = require("morgan");
7-
const path = require("path");
7+
const path = require("node:path");
88
const router = require("../lib");
99

1010
module.exports = (options, done) => {
@@ -32,7 +32,9 @@ module.exports = (options, done) => {
3232

3333
app.use(
3434
morgan("dev", {
35-
skip: (req, res) => options.quiet === "true" && res.statusCode < 400,
35+
skip: (req, res) =>
36+
req.baseUrl === "/healthcheck" ||
37+
(options.quiet === "true" && res.statusCode < 400),
3638
}),
3739
);
3840

src/static/bin/create.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,7 @@ $(() => {
178178
});
179179

180180
// fix type issues
181-
response.status = parseInt(response.status, 10);
181+
response.status = Number.parseInt(response.status, 10);
182182

183183
$('input[name="response"]').val(JSON.stringify(response));
184184
$("pre code").text(JSON.stringify(response, null, 2));

test/http/index.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
const cookieParser = require("cookie-parser");
22
const express = require("express");
33
const mockbin = require("../../lib");
4-
const path = require("path");
4+
const path = require("node:path");
55

66
const app = express();
77
let server = null;

0 commit comments

Comments
 (0)