Skip to content

Commit 8f5e0e6

Browse files
Potential fix for code scanning alert no. 26: Server-side request forgery
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
1 parent 7d7cd6d commit 8f5e0e6

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

src/app.controller.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -83,8 +83,12 @@ export class AppController extends AbstractController {
8383
@Param('project') project?: string,
8484
@Query('current') current?: string,
8585
): Promise<Response> {
86+
const validProjects = ['sesame-orchestrator', 'sesame-daemon', 'sesame-app-manager'];
87+
if (!validProjects.includes(project)) {
88+
throw new BadRequestException(`Invalid project: ${project}`);
89+
}
90+
8691
let data = <GithubUpdate>{};
87-
// console.log('this.storage', storage.get(project))
8892
if (storage.has(project)) {
8993
this.logger.log(`Fetching ${project} tags from cache`);
9094
data = storage.get(project) as GithubUpdate;
@@ -96,7 +100,6 @@ export class AppController extends AbstractController {
96100
data = await update.json();
97101
console.log('update', data)
98102
storage.set(project, data);
99-
// console.log('this.storage', storage.get(project))
100103
}
101104
// if (!Array.isArray(data)) {
102105
// throw new BadRequestException(`Invalid data from Github <${JSON.stringify(data)}>`);

0 commit comments

Comments
 (0)