Skip to content

Commit 44247c0

Browse files
committed
fix: check number type on inputs
1 parent de40fd3 commit 44247c0

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

src/services/creator.service.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,11 @@ export class CreatorScraper {
88
private person: CSFDCreator;
99

1010
public async creator(creatorId: number): Promise<CSFDCreator> {
11-
const url = creatorUrl(+creatorId);
11+
const id = Number(creatorId);
12+
if (isNaN(id)) {
13+
throw new Error('node-csfd-api: creatorId must be a valid number');
14+
}
15+
const url = creatorUrl(id);
1216
const response = await fetchPage(url);
1317

1418
const creatorHtml = parse(response);

src/services/movie.service.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,11 @@ export class MovieScraper {
2929
private film: CSFDMovie;
3030

3131
public async movie(movieId: number): Promise<CSFDMovie> {
32-
const url = movieUrl(+movieId);
32+
const id = Number(movieId);
33+
if (isNaN(id)) {
34+
throw new Error('node-csfd-api: movieId must be a valid number');
35+
}
36+
const url = movieUrl(id);
3337
const response = await fetchPage(url);
3438

3539
const movieHtml = parse(response);

0 commit comments

Comments
 (0)