PGL/Clyde
3
2
Fork 3
Code Issues 13 Pull Requests Actions Projects 3 Releases 2 Wiki Activity

fixed some issue for the token authorization #87

Merged
Maxime merged 2 commits from Max/Backend/GetUser into master 2024-03-10 23:28:14 +01:00
Conversation 3 Commits 2 Files Changed 6 +45 -13
6 changed files with 45 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 1ad67edabb - Show all commits

1
backend/src/main/java/ovh/herisson/Clyde/EndPoints/TokenController.java
View File

@ -10,7 +10,6 @@ import ovh.herisson.Clyde.Tables.Token;
@RestController
@CrossOrigin(origins = "http://localhost:5173")
tonitch commented 2024-03-09 21:13:05 +01:00
Review
Copy Link

Dans le dernier pr j'ai fait un petit changement au cross origin. Pour l'instant c'est bon mais c'est juste pour attirer l'attention sur ça !

Dans le dernier pr j'ai fait un petit changement au cross origin. Pour l'instant c'est bon mais c'est juste pour attirer l'attention sur ça !
public class TokenController {
private final TokenService tokenServ;

12
backend/src/main/java/ovh/herisson/Clyde/EndPoints/UserController.java
View File

@ -23,14 +23,10 @@ public class UserController {
}
@GetMapping("/user")
public ResponseEntity<User> getUser(@RequestHeader("Cookie") String cookie){
String[] tokens = cookie.split("=",2);
if (! tokens[0].equals("session_token") || tokens[1].length() != 64)
{
return new UnauthorizedResponse<>(null);
}
System.out.println(tokens[1]);
User user = authServ.getUserFromToken(tokens[1]);
public ResponseEntity<User> getUser(@RequestHeader("Cookie") String authorization){
tonitch commented 2024-03-09 21:10:55 +01:00
Outdated
Review
Copy Link

Pourquoi le passage à Coookie ?

Pourquoi le passage à Coookie ?
tonitch commented 2024-03-10 00:07:15 +01:00
Outdated
Review
Copy Link

frontend/src/rest/restConsumer.js Line 35 in 8fdfb470cb
'Authorization': session_token,

https://git.herisson.ovh/PGL/Clyde/src/commit/8fdfb470cb3c130f0d0e3bde1adc9b7e4edb1981/frontend/src/rest/restConsumer.js#L35
if (authorization == null) return new UnauthorizedResponse<>(null);
User user = authServ.getUserFromToken(authorization);
if (user == null) return new UnauthorizedResponse<>(null);
return new ResponseEntity<>(user, HttpStatus.OK);
}