Autentykacja (JWT), rejestracja i weryfikacja e-mail - Code review

Odpowiedz Nowy wątek
2019-05-07 21:26
0

Witam, proszę o CR projektu do nauki springa: https://github.com/tomaszp993/homebilling-java

Autentykację za pomocą JWT wykonałem według tego tutoriala: https://www.callicoder.com/sp[...]y-jwt-mysql-react-app-part-1/

@Edit Brak testów na obecną chwilę. Do dopisania ponowne wygenerowanie i wysłanie linku aktywacyjnego oraz reset hasła też na tokenach.

Dziękuję za poświęcony czas ;) Pozdrawiam :)

edytowany 4x, ostatnio: tomaszek993, 2019-05-07 21:36
Btw: uwierzytelnienie, nie autektykacja. - Patryk27 2019-05-07 23:02

Pozostało 580 znaków

2019-05-08 00:15
2

Nie znam się na Javie, ale na razie mam 2 punkty na minus:

  • Użycie słowa "autentykacja" (brr) zamiast "uwierzytelnianie"
  • Użycie JWT zamiast sesji do uwierzytelniania
@hauleth: Coś więcej dlaczego sesja, a nie JWT? - tomaszek993 2019-05-08 05:58

Pozostało 580 znaków

2019-05-08 13:31
3

-Podział warstwowy a nie per feature
-używanie wyjątków
-interface jak jest tylko jedna implementacja
-nadmiarowe javadoci

/**
     * Gets token type.
     *
     * @return the token type
     */
    public String getTokenType() {
        return TOKEN_TYPE;
    }

:)


Spring? Ja tam wole mieć kontrole nad kodem ᕙ(ꔢ)ᕗ
edytowany 1x, ostatnio: danek, 2019-05-08 13:31
@danek Biorę się za poprawki i doczytam o wyjątkach i podziale na warstwy. Czyli można przyjąć, że dla getterów i setterów javadoc można pominąć? - tomaszek993 2019-05-08 16:34
dawaj javadocki tam gdzie to faktycznie coś daje - danek 2019-05-18 00:52

Pozostało 580 znaków

Odpowiedz
Liczba odpowiedzi na stronę

1 użytkowników online, w tym zalogowanych: 0, gości: 1, botów: 0