IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

Développement Web en Java Discussion :

Bug au niveau de la méthode getPart ?


Sujet :

Développement Web en Java

  1. #1
    Membre éclairé
    Inscrit en
    Juin 2005
    Messages
    578
    Détails du profil
    Informations forums :
    Inscription : Juin 2005
    Messages : 578
    Par défaut Bug au niveau de la méthode getPart ?
    Bonjour

    J'ai créé une servlet qui a pour but de recevoir des données provenant d'un formulaire contenant un champ de type text et un champ de type file. J'ai aussi défini la taille maximum à ne dépasser dans mon web.xml:

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    <multipart-config>
        <location>/tmp</location>
        <max-file-size>20848820</max-file-size>
        <max-request-size>418018841</max-request-size>
        <file-size-threshold>1048576</file-size-threshold>
    </multipart-config>
    Le problème est que quand la taille du fichier dépasse la taille définie dans le web.xml, la valeur du champ de type text est remise à null.

    Pendant des mois j'ai pensé que j'avais mal fait les choses, jusqu'à ce que je tombe sur ça:

    With Servlet 3.0, if a MultipartConfig condition is violated (eg: maxFileSize), calling request.getParameter() returns null. Is this on purpose? What if I get some regular (text) parameters before calling getPart (and checking for an IllegalStateException)? This causes a NullPointerException to be thrown before I have a chance to check for the IllegalStateException
    Il s'agit d'une question posée par une personne ayant rencontré le même problème que moi, sur ce lien:

    https://stackoverflow.com/questions/...ng-jsp-servlet

    S'agit il d'un bug au niveau de la méthode getPart?

    La version de ma servlet est 3.1.

    Merci

  2. #2
    Membre éclairé
    Inscrit en
    Juin 2005
    Messages
    578
    Détails du profil
    Informations forums :
    Inscription : Juin 2005
    Messages : 578
    Par défaut
    Visiblement il s'agit d'un bug car je viens de tomber sur ça:

    However, if within the doPost method a request.getParameter([param_name]) is invoked on the request object BEFORE attempting to invoke request.getParts(), the Servlet will not throw an IllegalStateException if any of the parameters in the MultipartConfig have been exceeded. Instead, .getParts() returns a Collection of 0 parts, and without the Exception you have no idea why the multipart POST failed.

    If request.getParameter([param_name]) is invoked on the request object AFTER the first attempt to invoke request.getParts(), .getParts() will throw an IllegalStateException with the details about why the POST failed if any any of the parameters in the MultipartConfig have been exceeded.

    This took me quite a while to track down.

    The long and the short of it is that if you want to be able to check to make sure that any of the @MultipartConfig limitations are exceeded on the server-side (which you should) you MUST invoke request.getParts()before attempting to read any query string values.

    Certainly, there will be those that will say 'why not just check all of this on the client-side in JavaScript?'. To which I would say, that is absolutely what you should do. However, you should always double-check on the server-side to prevent someone who has managed to circumvent your client-side form validation. Furthermore, there are a number of very slick and easy JavaScript implementations to do this that do not work in various versions of IE.
    Il est dit qu'il faut d'abord faire appel getPart avant getParameter, je l'ai fait mais j'ai toujours le même problème:

    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
    42
    43
    44
    45
    46
    47
    48
    49
    50
    51
    52
    53
    54
    55
    56
    57
    58
    59
    60
    61
    62
    63
    64
    65
    66
    67
    68
    69
    70
    71
    72
    73
    74
    75
    76
    77
    78
    79
    80
    81
    82
    83
    84
    85
    86
    87
    88
    89
    90
    91
    92
    93
    94
    95
    96
    97
    98
    99
    100
    101
    102
    103
    104
    105
    106
    107
    108
    109
    110
    111
    112
    113
    114
    115
    116
    117
    118
    119
    120
    121
    122
    123
    124
    125
    126
    127
    128
    129
    130
    131
    132
    133
    134
    135
    136
    137
    138
    139
    140
    141
    142
    143
    144
    145
    146
    147
    148
    149
    150
    151
    152
    153
    154
    155
    156
    157
    158
    159
    160
    161
    162
    163
    164
    165
    166
    167
    168
    169
    170
    171
    172
    173
    174
    175
    176
    177
    178
    179
    180
    181
    182
    183
    184
    185
    186
    187
    188
    189
    190
    191
    192
    193
    194
    195
    196
    197
    198
    199
    200
    201
    202
    203
    204
    205
    206
    207
    208
    209
    210
    211
    212
    213
    214
    215
    216
    217
    218
    219
    220
    221
    222
    223
    224
    225
    226
    227
    228
    229
    230
    231
    232
    233
    234
    235
    236
    237
    238
    239
    240
    241
    242
    243
    244
    245
    246
    247
    248
    249
    250
    251
    252
    253
    254
    255
    256
    257
    258
    259
    260
    261
    262
    263
    264
    265
    266
    267
    268
    269
    270
    271
    272
    273
    274
    275
    276
    277
    278
    279
    280
    281
    282
    283
    284
    285
    286
    287
    288
    289
    290
    291
    292
    293
    294
    295
    296
    297
    298
    299
    300
    301
    302
    303
    304
    305
    306
    307
    308
    309
    310
    311
    312
    313
    314
    315
    316
    317
    318
    319
    320
    321
    /*
     * To change this template, choose Tools | Templates
     * and open the template in the editor.
     */
    package forms;
     
     
    import java.io.BufferedInputStream;
    import java.io.BufferedOutputStream;
    import java.io.File;
    import java.io.FileOutputStream;
    import java.io.IOException;
    import java.io.InputStream;
    import java.util.HashMap;
    import java.util.Map;
     
    import javax.servlet.ServletException;
    import javax.servlet.http.HttpServletRequest;
    import javax.servlet.http.Part;
     
    import beans.Journal;
    import dao.JournalDao;
    import dao.DAOException;
     
    import eu.medsea.mimeutil.MimeUtil;
     
    public final class CreationJournalForm {
        private static final String CHAMP_ANNEE       = "annee";
        private static final String CHAMP_MOIS   = "mois";
        private static final String CHAMP_FICHIER   = "fichier";
     
        private static final int    TAILLE_TAMPON   = 10240;                        // 10ko
     
        private String              resultat;
        private Map<String, String> erreurs         = new HashMap<String, String>();
        private JournalDao           journalDao;
     
        public CreationJournalForm( JournalDao journalDao ) {
            this.journalDao = journalDao;
        }
     
        public Map<String, String> getErreurs() {
            return erreurs;
        }
     
        public String getResultat() {
            return resultat;
        }
     
        public Journal creerJournal( HttpServletRequest request, String chemin )  {
     
     
     
     
          //  String categorie = "batik";
         //  String prix = "11";
           // String description = "blablablabla";
           // String img = getValeurChamp( request, CHAMP_IMAGE);
          // System.out.println("test "+img);
            Journal journal = new Journal();
     
            traiterFichier(journal, request, chemin, "2018", "01");
            String annee = getValeurChamp( request, CHAMP_ANNEE );
            String mois = getValeurChamp( request, CHAMP_MOIS );
            traiterAnnee(annee,mois, journal);
            traiterMois(mois, journal);
     
     
     
            try {
                if ( erreurs.isEmpty() ) {
                    journalDao.creer( journal );
                    resultat = "Ajout du fichier réussi!";
                } else {
                    resultat = "Échec de la création du journal informatique.";
                }
            } catch ( DAOException e ) {
                setErreur( "imprévu", "Erreur imprévue lors de la création." );
                resultat = "Échec de la création du journal informatique : une erreur imprévue est survenue, merci de réessayer dans quelques instants.";
                e.printStackTrace();
                System.out.println(e);
            }
     
            return journal;
        }
     
        private void traiterAnnee( String annee,String mois, Journal journal ) {
            try {
                validationAnnee( annee );
            } catch ( FormValidationException e ) {
                setErreur( CHAMP_ANNEE, e.getMessage() );
            }
            journal.setAnnee(annee+"-"+mois+"-01");
        }
     
       private void traiterMois( String mois, Journal journal ) {
            try {
                validationMois( mois );
            } catch ( FormValidationException e ) {
                setErreur( CHAMP_MOIS, e.getMessage() );
            }
            journal.setMois(mois );
        }
     
        private void traiterFichier( Journal journal, HttpServletRequest request, String chemin, String annee, String mois ) {
            String fichier = null;
            try {
                fichier = validationFichier( request, chemin, annee, mois );
            } catch ( FormValidationException e ) {
                setErreur( CHAMP_FICHIER, e.getMessage() );
            }
            journal.setFichier( fichier );
        }
     
        private void validationAnnee( String annee ) throws FormValidationException {
            if ( annee == null  ) {
                throw new FormValidationException( "Veuillez entrer l'année." );
            } 
     
        }
     
         private void validationMois( String mois ) throws FormValidationException {
     
            if ( mois.equals("choix")  ) {
                throw new FormValidationException( "Merci de choisir le mois." );
            } 
        }
     
        private String validationFichier( HttpServletRequest request, String chemin, String annee, String mois ) throws FormValidationException {
            /*
             * Récupération du contenu du champ image du formulaire. Il faut ici
             * utiliser la méthode getPart().
             */
            String nomFichier = null;
            InputStream contenuFichier = null;
     
            try {
              Part   part = request.getPart( CHAMP_FICHIER );
              nomFichier = getNomFichier( part );
    System.out.println("nom fichier est: "+ nomFichier);
                /*
                 * Si la méthode getNomFichier() a renvoyé quelque chose, il s'agit
                 * donc d'un champ de type fichier (input type="file").
                 */
                if ( nomFichier != null && !nomFichier.isEmpty() ) {
                    /*
                     * Antibug pour Internet Explorer, qui transmet pour une raison
                     * mystique le chemin du fichier local à la machine du client...
                     * 
                     * Ex : C:/dossier/sous-dossier/fichier.ext
                     * 
                     * On doit donc faire en sorte de ne sélectionner que le nom et
                     * l'extension du fichier, et de se débarrasser du superflu.
                     */
                    nomFichier = nomFichier.substring( nomFichier.lastIndexOf( '/' ) + 1 )
                            .substring( nomFichier.lastIndexOf( '\\' ) + 1 );
     
                    /* Récupération du contenu du fichier */
                    contenuFichier = part.getInputStream();
     
                    /* Extraction du type MIME du fichier depuis l'InputStream */
                    MimeUtil.registerMimeDetector( "eu.medsea.mimeutil.detector.MagicMimeMimeDetector" );
                    String contentType = request.getServletContext().getMimeType( nomFichier );
     
                    /*
                     * Si le fichier est bien une image, alors son en-tête MIME
                     * commence par la chaîne "image"
                     */
                    if ( contentType.startsWith( "application/pdf" ) ) {
                        /* Écriture du fichier sur le disque */
     
                        String[] fileNameSplits = nomFichier.split("\\.");
                        int extensionIndex = fileNameSplits.length - 1;
                        nomFichier = annee+mois + "." + fileNameSplits[extensionIndex];
                        System.out.println("fichier est: "+nomFichier);
     
                        ecrireFichier( contenuFichier, nomFichier, chemin );
                    } else {
                        throw new FormValidationException( "Le fichier envoyé doit être un pdf." );
                    }
                }  
     
     
            }
     
             catch ( NullPointerException e ) {
                /*
                 * Exception retournée si la taille des données dépasse les limites
                 * définies dans la section <multipart-config> de la déclaration de
                 * notre servlet d'upload dans le fichier web.xml
                 */
               //e.printStackTrace();
                throw new FormValidationException( "Le fichier ne doit pas être vide.." );
            }
     
            catch ( IllegalStateException e ) {
                /*
                 * Exception retournée si la taille des données dépasse les limites
                 * définies dans la section <multipart-config> de la déclaration de
                 * notre servlet d'upload dans le fichier web.xml
                 */
               //e.printStackTrace();
                throw new FormValidationException( "Le fichier envoyé ne doit pas dépasser 2Mo." );
            }
     
            catch ( IOException e ) {
                /*
                 * Exception retournée si une erreur au niveau des répertoires de
                 * stockage survient (répertoire inexistant, droits d'accès
                 * insuffisants, etc.)
                 */
                e.printStackTrace();
                throw new FormValidationException( "Erreur de configuration du serveur." );
            } catch ( ServletException e ) {
                /*
                 * Exception retournée si la requête n'est pas de type
                 * multipart/form-data.
                 */
                e.printStackTrace();
                throw new FormValidationException(
                        "Ce type de requête n'est pas supporté, merci d'utiliser le formulaire prévu pour envoyer votre fichier." );
            }
     
            return nomFichier;
        }
     
     
        /*
         * Ajoute un message correspondant au champ spécifié à la map des erreurs.
         */
        private void setErreur( String champ, String message ) {
            erreurs.put( champ, message );
        }
     
        /*
     
     * Méthode utilitaire qui a pour unique but de lire l'InputStream contenu
     
     * dans l'objet part, et de le convertir en une banale chaîne de caractères.
     
     */
     
     
     
        /*
         * Méthode utilitaire qui retourne null si un champ est vide, et son contenu
         * sinon.
         */
         private static String getValeurChamp( HttpServletRequest request, String nomChamp ) {
            String valeur = request.getParameter( nomChamp );
            if ( valeur == null || valeur.trim().length() == 0 ) {
                return null;
            } else {
                return valeur;
            }
        }
     
        /*
         * Méthode utilitaire qui a pour unique but d'analyser l'en-tête
         * "content-disposition", et de vérifier si le paramètre "filename" y est
         * présent. Si oui, alors le champ traité est de type File et la méthode
         * retourne son nom, sinon il s'agit d'un champ de formulaire classique et
         * la méthode retourne null.
         */
        private static String getNomFichier( Part part ) {
            /* Boucle sur chacun des paramètres de l'en-tête "content-disposition". */
            for ( String contentDisposition : part.getHeader( "content-disposition" ).split( ";" ) ) {
                /* Recherche de l'éventuelle présence du paramètre "filename". */
                if ( contentDisposition.trim().startsWith( "filename" ) ) {
                    /*
                     * Si "filename" est présent, alors renvoi de sa valeur,
                     * c'est-à-dire du nom de fichier sans guillemets.
                     */
                    return contentDisposition.substring( contentDisposition.indexOf( '=' ) + 1 ).trim().replace( "\"", "" );
                }
            }
            /* Et pour terminer, si rien n'a été trouvé... */
            return null;
        }
     
        /*
         * Méthode utilitaire qui a pour but d'écrire le fichier passé en paramètre
         * sur le disque, dans le répertoire donné et avec le nom donné.
         */
        private void ecrireFichier( InputStream contenuFichier, String nomFichier, String chemin )
                throws FormValidationException {
            /* Prépare les flux. */
            BufferedInputStream entree = null;
            BufferedOutputStream sortie = null;     
     
            try {
                /* Ouvre les flux. */
                entree = new BufferedInputStream( contenuFichier, TAILLE_TAMPON );
                sortie = new BufferedOutputStream( new FileOutputStream( new File( chemin + nomFichier ) ),
                        TAILLE_TAMPON );
     
                /*
                 * Lit le fichier reçu et écrit son contenu dans un fichier sur le
                 * disque.
                 */
                byte[] tampon = new byte[TAILLE_TAMPON];
                int longueur = 0;
                while ( ( longueur = entree.read( tampon ) ) > 0 ) {
                    sortie.write( tampon, 0, longueur );
                }
            } catch ( Exception e ) {
                throw new FormValidationException( "Erreur lors de l'écriture du fichier sur le disque." );
            } finally {
                try {
                    sortie.close();
                } catch ( IOException ignore ) {
     
                }
                try {
                    entree.close();
                } catch ( IOException ignore ) {
     
                }
            }
        }
    }
    J'ai aussi lu quelque part qu'à cause de ce problème, il est préférable d'utiliser jquery pour la gestion de la taille du fichier.

Discussions similaires

  1. Réponses: 6
    Dernier message: 22/05/2008, 09h13
  2. BUGs au niveau d'une application access
    Par monsesilo dans le forum IHM
    Réponses: 1
    Dernier message: 14/06/2007, 10h29
  3. Réponses: 11
    Dernier message: 11/11/2006, 12h20

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo