Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Snyk] Upgrade express from 4.18.2 to 5.0.0 #285

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1,010 changes: 548 additions & 462 deletions authentication/package-lock.json

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion authentication/package.json
Original file line number Diff line number Diff line change
@@ -19,7 +19,7 @@
"cookie-parser": "^1.4.6",
"cors": "^2.8.5",
"dotenv": "^16.3.1",
"express": "^4.18.2",
"express": "^5.0.0",
"express-useragent": "^1.0.15",
"google-auth-library": "^9.4.1",
"helmet": "^7.0.0",

Unchanged files with check annotations Beta

throw new Error('Refresh token is revoked.');
}
// TODO: check if userId is in sql database

Check warning on line 161 in authentication/src/services/auth.service.ts

GitHub Actions / lint

Unexpected 'todo' comment: 'TODO: check if userId is in sql database'
return this.jwtHandler.createToken(
decoded.userId,
};
addPost(newPost);
// TODO: Add actual API call to add post to database

Check warning on line 59 in client/src/pages/posts/components/create/Create.tsx

GitHub Actions / lint

Unexpected 'todo' comment: 'TODO: Add actual API call to add post to...'
toast({ title: "Post Created Successfully", status: "success" });
navigate("/posts");
};
};
const refreshPost = async () => {
// TODO: Add actual API call to get post by id from database

Check warning on line 55 in client/src/pages/posts/components/edit/Edit.tsx

GitHub Actions / lint

Unexpected 'todo' comment: 'TODO: Add actual API call to get post by...'
};
useEffect(() => {
};
updatePost(newPost);
// TODO: Add actual API call to edit post in database

Check warning on line 87 in client/src/pages/posts/components/edit/Edit.tsx

GitHub Actions / lint

Unexpected 'todo' comment: 'TODO: Add actual API call to edit post...'
toast({ title: "Post updated Successfully", status: "success" });
navigate("/posts");
};
const columns: ColumnDef<IPost>[] = usePostColumns();
const fetchAllPosts = async () => {
// TODO: Replace the following with actual API call

Check warning on line 50 in client/src/pages/posts/components/list/List.tsx

GitHub Actions / lint

Unexpected 'todo' comment: 'TODO: Replace the following with actual...'
const data: IPost[] = makeData(100);
setPosts(data);
};
};
const onDelete = () => {
// TODO: Add actual API call to remove post from database

Check warning on line 41 in client/src/pages/posts/components/list/PostColumns.tsx

GitHub Actions / lint

Unexpected 'todo' comment: 'TODO: Add actual API call to remove post...'
removePost(activePopoverId as number);
toast({ title: `Post deleted ${activePopoverId}`, status: "error" });
setActivePopoverId(null);
};
const fetchAllCategories = async () => {
// TODO: Replace with actual API call

Check warning on line 48 in client/src/pages/posts/components/list/PostColumns.tsx

GitHub Actions / lint

Unexpected 'todo' comment: 'TODO: Replace with actual API call'
setCategories(getAllCategories);
};
const { id } = useParams<{ id: string }>();
const refreshPost = async () => {
// TODO: Add actual API call to get post by id from database

Check warning on line 35 in client/src/pages/posts/components/show/Show.tsx

GitHub Actions / lint

Unexpected 'todo' comment: 'TODO: Add actual API call to get post by...'
};
useEffect(() => {
import applicationPermissions from "~models/Permissions";
class PermissionService {
// TODO: singleton instance

Check warning on line 6 in client/src/shared/services/Permission.service.ts

GitHub Actions / lint

Unexpected 'todo' comment: 'TODO: singleton instance'
constructor() {
this.getCurrentUserRoles = this.getCurrentUserRoles.bind(this);
this.verifyRole = this.verifyRole.bind(this);