Discussion :

[Népomucène]

Bonjour

Dans une application JSF sur Glassfish, pour enregistrer les erreurs inattendues, j'ai utilisé un vieux truc tout simple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
OutputStream stream ...
// sachant que stream va écrire dans un fichier texte
 
System.setErr(new PrintStream(stream));

ça marche impec et j'ai bien toutes les erreurs non gérées dans mon fichier texte.

Le problème :
Si j'ai plusieurs applications dans GlassFish (war), alors les erreurs des différentes applications se retrouvent mélangées dans un seul fichier texte.
A l'évidence, System.setErr s'applique à toutes les applications déployées sur le serveur.
J'aurai voulu avoir un fichier d'erreur par application

Du coup j'ai essayé avec java.util.logging.Logger et Log4j mais ils ne semblent pas gérer les erreurs inattendues (ou alors j'ai pas trouvé)

Quelqu'un a-t-il une idée ?

[OButterlin]

Dans la mesure où toutes les requêtes JSF passent par la servlet FacesServlet, il devrait être possible de surcharger la méthode service pour mettre un logger dans le cas d'une erreur imprévue.

[Népomucène]

surcharger la méthode service pour mettre un logger dans le cas d'une erreur imprévue.

Au-dessus de mes compétences ; j'ignorai l'existence de cette classe.

Pour surcharger la méthode, je suppose qu'il faudrait créer une classe qui hérite de FacesServlet mais j'ai cru voir dans la doc que c'est une classe "final".
Il faudra aussi modifier le web.xml et indiquer la nouvelle classe ?
J'ai l'impression que je suis complètement à côté.

[OButterlin]

Effectivement, la classe est marquée "final", donc on ne peut en pas hériter.

Ceci dit, on peut facilement contourner le problème en faisant une servlet héritant de HttpServlet et possédant une propriété de type FacesServlet.
Ensuite, on définit toutes les méthodes publiques de FacesServlet dans notre servlet frontale et on redirige vers les méthodes de FacesServlet.

Pour l'activation, tout se passe dans le fichier web.xml, il suffit de faire référence à la nouvelle servlet en remplacement de javax.faces.FacesServlet.

J'ai fait le test, ça fonctionne... mais ce n'est pas vraiment intéressant. Pour arriver dans le traitement d'une exception au niveau de la méthode service, il faut faut vraiment y aller !
Et on a déjà pollué les logs avec des stackTrace.
Il y a certainement un point plus approprié pour faire des traces propres, je vais "éplucher" le cycle pour voir ce qu'on peut faire

[Népomucène]

Merci à toi pour cette recherche.
Si c'est trop galère, n'insiste pas ; je vais logger déjà tout ce que je peux comme erreur
et je ferai un log général avec System.setErr que j'examinerai régulièrement pour arriver à éliminer un max d'erreurs imprévues

[OButterlin]

J'ai trouvé plusieurs possibilités mais à priori, la plus simple dans ton cas serait de passer par une "custom" ExceptionHandlerFactory.
En gros, on a :
1- une classe CustomExceptionHandlerFactory étendant ExceptionHandlerFactory et faisant référence à CustomExceptionHandler
2- une classe CustomExceptionHandler étendant ExceptionHandlerWrapper qui fait la log "propre".

(on suppose que ces classes sont dans un package com.olivier.jsf.factory)

CustomExceptionHandlerFactory

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
public class CustomExceptionHandlerFactory extends ExceptionHandlerFactory
{
    private ExceptionHandlerFactory exceptionHandlerFactory;
 
    public CustomExceptionHandlerFactory()
    {
    }
 
    public CustomExceptionHandlerFactory(ExceptionHandlerFactory exceptionHandlerFactory)
    {
        this.exceptionHandlerFactory = exceptionHandlerFactory;
    }
 
    @Override
    public ExceptionHandler getExceptionHandler()
    {
        ExceptionHandler handler = new CustomExceptionHandler(exceptionHandlerFactory.getExceptionHandler());
        return handler;
    }
}

CustomExceptionHandler

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
public class CustomExceptionHandler extends ExceptionHandlerWrapper
{
    private static final Logger logger = Logger.getLogger(CustomExceptionHandler.class.getName());
    private ExceptionHandler exceptionHandler;
 
    public CustomExceptionHandler(ExceptionHandler exceptionHandler)
    {
        this.exceptionHandler = exceptionHandler;
    }
 
    @Override
    public ExceptionHandler getWrapped()
    {
        return exceptionHandler;
    }
 
    @Override
    public void handle() throws FacesException
    {
        final Iterator<ExceptionQueuedEvent> queue = getUnhandledExceptionQueuedEvents().iterator();
        while (queue.hasNext())
        {
            // take exceptions one by one
            ExceptionQueuedEvent item = queue.next();
            ExceptionQueuedEventContext exceptionQueuedEventContext = (ExceptionQueuedEventContext) item.getSource();
            try
            {
                // log error
                Throwable throwable = exceptionQueuedEventContext.getException();
                logger.log(Level.SEVERE, "EXCEPTION: " + throwable.toString());
            }
            finally
            {
                // remove it from queue
                queue.remove();
            }
        }
        getWrapped().handle();
    }
 
}

Mise en oeuvre dans faces-config.xml

Code XML :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
<?xml version="1.0" encoding="UTF-8"?>
<faces-config
    xmlns="http://xmlns.jcp.org/xml/ns/javaee"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
    xsi:schemaLocation="http://xmlns.jcp.org/xml/ns/javaee http://xmlns.jcp.org/xml/ns/javaee/web-facesconfig_2_3.xsd"
    version="2.3">
 
    <factory>
        <exception-handler-factory>com.olivier.jsf.factory.CustomExceptionHandlerFactory</exception-handler-factory>
    </factory>
 
    <application>
...