Projet

Général

Profil

Bug #10416

La configuration est en JSON, donc les chaînes en unicode et il arrive qu'elles soit combinées avec des chaînes simple contenant des caractères non-ASCII

Ajouté par Benjamin Dauvergne il y a environ 8 ans. Mis à jour il y a plus de 7 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
24 mars 2016
Echéance:
% réalisé:

100%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

La configuration devrait être en grand partie simplifiée en chaînes simples.


Fichiers

Révisions associées

Révision 3677cd0c (diff)
Ajouté par Benjamin Dauvergne il y a environ 8 ans

flatten some config string to str (fixes #10416)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

Ce patch devrait corriger cette trace:

>   File "usr/share/pyshared/mandaye/dispatcher.py", line 141, in Dispatcher.set_request_target
>     self = <mandaye.dispatcher.Dispatcher object at 0x325d050>
>     request = <HTTPRequest {'msg': None, 'headers': {'accept-language': ['fr-...
>     139 """ Add target url on not into the request
>     140 """ 
> --> 141 request.target = self.target.geturl() + self.env['RAW_URI']
>     142 if self.req_mapping['target']:
>     143     if "//" in self.req_mapping['target']:
> UnicodeDecodeError: 'ascii' codec can't decode byte 0xe9 in position 86: ordinal not in range(128)
#2

Mis à jour par Serghei Mihai il y a environ 8 ans

Pourquoi tu reboucles sur les parametres de config au lieu d'utiliser la boucle existante, genre:

                    for param in ['site_name', 'location', 'target', 'server_name', 'mapper', 'auth_type']:
                        if not conf.has_key(param):
                            error = 'you must set %s option in vhost : %s' % \
                                    (param, conf_file)
                            logger.error(error)
                            raise ImproperlyConfigured, error
                        conf[param] = strize(conf[param]) <---

#3

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

Parce que si tu regardes bien la liste de paramètre n'est pas la même, j'ai enlevé site_name qui doit rester en unicode.

#4

Mis à jour par Serghei Mihai il y a environ 8 ans

Ok, go

#5

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

  • Statut changé de Nouveau à Résolu (à déployer)
  • % réalisé changé de 0 à 100

Appliqué par commit commit:3677cd0c1952b27e6762e4b174b225bd8b0d7549.

#6

Mis à jour par Benjamin Dauvergne il y a plus de 7 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF