Reusinage de ‘RequestHandler::doRequest’
La méthode RequestHandler::doRequest
est excessivement longue, non testé, et a de multiple responsabilités:
- Association de la
request-map
au handler - Benchmarking
- Suivi des utilisateur
- Gestion de HTTPS
- Gestion des redirection vers d'autres routes
Ce qui la rend particulièrement difficile à lire est que c'est un gruyère de tests à null et de traitements d'erreurs, avec des variables mutées déclarées 200 lignes plus haut.
Il faudrait réusiner cette méthode en améliorant le traitement d'erreur et en déléguant les responsabilités à d'autres méthodes.
-
Tester unitairement cette méthode -
Extraire le suivi des utilisateurs -
Extraire la mesure de temps -
Extraire la résolution de la request-map
(OFBIZ-10438) -
Extraire la vérification des certificats SSL (OFBIZ-10450) -
Utiliser Optional
quand cela est possible.