CHANGELOG.md

@@ -18,7 +18,16 @@
 -->
 
 
-## 4.0.0 (???)
+## 4.1.0 (Upcoming)
+
+### ✨ Feature
+- Add features related to corrige (commentary, commit specific link / update, delete link)
+
+### 🎨 Interface
+- Ask for confirmation before creating an exercise that already exists
+
+
+## 4.0.0 (2024-04-19)
 
 ### ✨ Feature
 - Add features related to corrige (commentary, commit specific link / update, delete link)

ExpressAPI/package-lock.json

 {
     "name": "dojo_backend_api",
-    "version": "4.0.0",
+    "version": "4.1.0",
     "lockfileVersion": 3,
     "requires": true,
     "packages": {
         "": {
             "name": "dojo_backend_api",
-            "version": "4.0.0",
+            "version": "4.1.0",
             "license": "AGPLv3",
             "dependencies": {
                 "@dotenvx/dotenvx": "^0.27.1",

ExpressAPI/package.json

 {
     "name"           : "dojo_backend_api",
     "description"    : "Backend API of the Dojo project",
-    "version"        : "4.0.0",
+    "version"        : "4.1.0",
     "license"        : "AGPLv3",
     "author"         : "Michaël Minelli <dojo@minelli.me>",
     "main"           : "dist/src/app.js",

ExpressAPI/src/middlewares/SecurityMiddleware.ts

@@ -6,8 +6,8 @@ import AssignmentManager from '../managers/AssignmentManager.js';
 
 
 class SecurityMiddleware {
-    private isConnected(checkIfConnected: boolean, req: express.Request): boolean {
-        return checkIfConnected && req.session.profile !== null && req.session.profile !== undefined;
+    private checkIfConnected(checkIfConnected: boolean, req: express.Request): boolean {
+        return !checkIfConnected || (req.session.profile !== null && req.session.profile !== undefined);
     }
 
     private async checkType(checkType: SecurityCheckType, req: express.Request): Promise<boolean> {
@@ -33,7 +33,7 @@ class SecurityMiddleware {
     // First check if connected then check if at least ONE rule match. It's NOT an AND but it's a OR function.
     check(checkIfConnected: boolean, ...checkTypes: Array<SecurityCheckType>): (req: express.Request, res: express.Response, next: express.NextFunction) => void {
         return (req: express.Request, res: express.Response, next: express.NextFunction) => {
-            if ( !this.isConnected(checkIfConnected, req) ) {
+            if ( !this.checkIfConnected(checkIfConnected, req) ) {
                 return req.session.sendResponse(res, StatusCodes.UNAUTHORIZED);
             }