Project

General

Profile

Tâche #8933

Scénario #10250: Autoriser les caractères spéciaux dans les adresses des sites interdits et autorisés

caractère interdit dans l'url

Added by Christophe Dezé over 6 years ago. Updated about 6 years ago.

Status:
Fermé
Priority:
Normal
Assigned To:
Start date:
09/17/2014
Due date:
01/22/2015
% Done:

100%

Estimated time:
3.00 h
Spent time:
Remaining (hours):
0.0

Description

Nous avions demandé la possibilité de renseigner des url dans l'ead pour autoriser des sites.
http://dev-eole.ac-dijon.fr/issues/5493

cela a été fait mais ces urls ne doivent pas contenir certains cratères car ils font planter le formulaire dans l'EAD
http://dev-eole.ac-dijon.fr/issues/4355

c'est ballot d'interdire des caractères arbitrairement, au lieu de gérer correctement l'encodage des url dans l'EAD, pour que s'affiche correctement le formulaire!
peut-on réautoriser '?', '=', '&', '%', '"', "'" ?
merci

replace24.patch View (1.39 KB) Joël Cuissinat, 01/16/2015 11:40 AM

Associated revisions

Revision 7fe71f61 (diff)
Added by Joël Cuissinat about 6 years ago

Caractères '?', '=' et '&' réautorisés pour les URL interdites/autorisées

Fixes: #8933 @2h

History

#1 Updated by Christophe Dezé over 6 years ago

j'ai enlevé le ? et le = des caractères interdits et j'ai pu rajouter une url de video youtube sans soucis
http://dev-eole.ac-dijon.fr/issues/8933

# caractères interdits dans les adresses à filter
for char in ( '&', '%', '"', "'"):
if char in site:
raise Exception("""Erreur :\\nLe caractère %s n'est pas autorisé dans les adresses""" % char)

#2 Updated by Christophe Dezé over 6 years ago

Christophe DEZE a écrit :

j'ai enlevé le ? et le = des caractères interdits et j'ai pu rajouter une url de video youtube sans soucis
http://dev-eole.ac-dijon.fr/issues/8933

# caractères interdits dans les adresses à filter
for char in ( '&', '%', '"', "'"):
if char in site:
raise Exception("""Erreur :\\nLe caractère %s n'est pas autorisé dans les adresses""" % char)

dans /usr/share/ead2/backend/actions/amon/forms.py

#3 Updated by Luc Bourdot about 6 years ago

  • Tracker changed from Anomalie to Scénario
  • Due date set to 01/22/2015
  • Target version set to sprint 2015 2-4
  • Release set to Mises à jour 2.4.1
  • Story points set to 3.0
  • Distribution changed from EOLE 2.3 to EOLE 2.4

#4 Updated by Joël Cuissinat about 6 years ago

  • Tracker changed from Scénario to Tâche
  • Target version deleted (sprint 2015 2-4)
  • Estimated time set to 3.00 h
  • Remaining (hours) set to 3.0
  • Distribution changed from EOLE 2.4 to Toutes

#5 Updated by Joël Cuissinat about 6 years ago

  • Parent task set to #10250

#6 Updated by Joël Cuissinat about 6 years ago

  • Status changed from Nouveau to Résolu
  • % Done changed from 0 to 100

#7 Updated by Joël Cuissinat about 6 years ago

  • Assigned To set to Joël Cuissinat
  • Remaining (hours) changed from 3.0 to 0.5

#8 Updated by Joël Cuissinat about 6 years ago

Correction a minima mais fonctionnelle (2.3 & 2.4).
Je joins un patch qui contient un début d'implémentation pour le support de la séquence "%22" sur 2.4 mais je ne pense pas qu'on reviennent sur le sujet dans le cadre de l'EAD2).

#9 Updated by Emmanuel GARETTE about 6 years ago

  • Status changed from Résolu to Fermé
  • Remaining (hours) changed from 0.5 to 0.0

Testé sur 2.4.

Sont autorisés :

Les urls suivantes :

Sont remplacés par :

sans avertissement complémentaire.

#10 Updated by Joël Cuissinat about 6 years ago

  • Status changed from Fermé to En cours
  • % Done changed from 100 to 80
  • Remaining (hours) changed from 0.0 to 0.5

#11 Updated by Joël Cuissinat about 6 years ago

Emmanuel GARETTE a écrit :

Les urls suivantes :

Sont remplacés par :

sans avertissement complémentaire.

C'est pas ce que j'avais prévu !

#12 Updated by Joël Cuissinat about 6 years ago

  • Status changed from En cours to Fermé
  • Remaining (hours) changed from 0.5 to 0.0

La demande initiale a été résolue (l'interdiction d'URL d'accès à des vidéos youtube est possible).

Le dernier problème soulevé est lié à d'autres modifications et devra être traité de façon plus globale => #10324

#13 Updated by Joël Cuissinat about 6 years ago

  • % Done changed from 80 to 100

Also available in: Atom PDF