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

fixed a token encoding issue #70

Merged
tonitch merged 3 commits from Max/backend/loginFix into master 2024-03-09 09:57:54 +01:00
Conversation 1 Commits 3 Files Changed 2 +1 -5
1 changed files with 1 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 521c98fdc2 - Show all commits

6
backend/src/main/java/ovh/herisson/Clyde/Services/TokenService.java
View File

@ -25,11 +25,7 @@ public class TokenService {
byte[] bytes = new byte[64];
new SecureRandom().nextBytes(bytes);
Maxime marked this conversation as resolved
tonitch commented 2024-03-08 20:53:57 +01:00
Review
Copy Link

je suis un peu curieux de la différence avec Random()

je suis un peu curieux de la différence avec Random()
Maxime commented 2024-03-08 20:58:06 +01:00
Review
Copy Link

random est moins random et plus prévisible donc plus de conflits (on m'a juste conseillé SecureRandom donc sur un site donc voila
)

random est moins random et plus prévisible donc plus de conflits (on m'a juste conseillé SecureRandom donc sur un site donc voila )
for (int i = 0; i < bytes.length; i++) {
while (bytes[i] == 0){
byte[] temp = new byte[1];
new SecureRandom().nextBytes(temp);
bytes[i] = temp[0];
}
bytes[i] = (byte) (((bytes[i]+256)%256 %95+ 32));
tonitch commented 2024-03-08 21:14:51 +01:00
Outdated
Review
Copy Link

tu peux aussi ( x / 4 (division entière) + 64) et ainsi tu est dans une range affichable (lazy)

tu peux aussi ( x / 4 (division entière) + 64) et ainsi tu est dans une range affichable (lazy)
Maxime commented 2024-03-08 23:51:38 +01:00
Outdated
Review
Copy Link

smart je vais try

smart je vais try
Maxime commented 2024-03-09 09:38:32 +01:00
Review
Copy Link

Permet d'avoir des caractères affichable

Permet d'avoir des caractères affichable
}
// will never end up in the catch because of the way that SecureRandom.nextBytes is implemented
try {