myunimol / api

API for MyUnimol
https://myunimol.it
11 stars 2 forks source link

Semplificazione struttura #1

Closed mattmezza closed 9 years ago

mattmezza commented 9 years ago

Ho semplificato la struttura delle servlet

Ho creato in it.unimol.my.utils.WebServiceServlet una servlet che blocca le richieste in GET. Ho definito in questa classe un metodo che controlla il token. Ho creato in it.unimol.my.utils.Esse3AuthServlet una servlet che controlla le credenziale (nel senso che controlla se sono o meno nulle) e ho definito un metodo per questo. Ho integrato i cambiamenti in alcune servlet.

Questo cambiamento ha fatto sì che l'implementazione di una servlet si riduce al solo override del metodo doPost(...). Il controllo del token si riduce a

if(!tokenIsValid(req,rest)) {
    return;
}

subito dopo aver chiamato super.doPost(...) nel metodo doPost(...). Analogamente si procede per il controllo delle credenziali (il metodo è credentialsAreOk(...))

Spero questo possa semplificare il lavoro.

per qualsiasi richiesta commentate qui sotto.
mattmezza commented 9 years ago

Ah ho spostato anche delle variabili di istanza in comune a tutte le servlet dei webservices nella classe da cui ereditato tutte le servlet myunimol. Per maggiori dettagli controllate it.unimol.my.utils.WebServiceServlet :+1:

R0adRunn3r commented 9 years ago

Perfetto domani mi vedo tutte le modifiche che hai apportato ;) Il 08/dic/2014 19:16 "Matteo Merola" notifications@github.com ha scritto:

Ah ho spostato anche delle variabili di istanza in comune a tutte le servlet dei webservices nella classe da cui ereditato tutte le servlet myunimol. Per maggiori dettagli controllate it.unimol.my.utils.WebServiceServlet [image: :+1:]

— Reply to this email directly or view it on GitHub https://github.com/cbranca/myunimol-webservices/issues/1#issuecomment-66159249 .

mattmezza commented 9 years ago

Ho semplificato ridicolmente la struttura: ora basta estendere Esse3AuthServlet e fare Override di serve(...) per avere una servlet che automaticamente valida il token e controlla che username e password non siano null. Per maggiori dettagli guardate una servlet qualsiasi.