Pas de limite du nombre de caractères sur la recherche

[gustavi]

Aujourd'hui on peut faire des recherches sans limitation dans le nombre de caractères. Il faut une limite, je pense qu'entre 64 et 256 c'est bien.

[GerardPaligot]

Pourquoi ?

[gustavi]

Pourquoi quoi ?

La limite ? Aujourd'hui tu peux envoyer une requête avec 500Mo de caractères. Ça me parait pas être vraiment quelque chose à faire.

[Situphen]

Tu veux limiter comment ? Si tu veux limiter le nombre de caractères que l'utilisateur peut rentrer dans la barre de recherche, j'en vois pas l'intérêt car il suffit d'ouvrir la console pour l'enlever (ou d'utiliser un logiciel/script).

Si c'est côté serveur, oui pourquoi pas. Après la recherche utilise une méthode GET, donc ça passe par l'URL et la taille d'un lien est limité je crois (j'ai testé sur Google au bout d'un moment j'ai une erreur 400).

[gustavi]

Une limitation front est une bonne chose mais comme tu l'as dit inutile si on utilise une console ou autre.

Côté back ça me semble nécessaire.

[squik]

bonjour je suis du groupe d'étudiants j'ai partiellement corrigé le front avec le maxlength , il me reste que la barre de recherche avec le {{ form.q }} dans le templates/search/search.html qui me pose un problème, je ne sais pas où le modifier de plus c'est le seul qui est touché par l'url ,on pourrait envoyer une page d'erreur quand le get donne un requête de plus de 256 caractères.

merci d'avance pour votre aide.

[gustavi]

Si il y a un « maxlenght » côté Django alors au niveau du champs rien à faire, c'est généré automatiquement.

[pierre-24]

Mmmmh, j'ai peut être oublié ça :)

[Situphen]

Bon, il y a bien le maxlengthqui bloque à 150 caractères ! Si je l'enlève avec la console et que j'envoie le formulaire j'ai une belle erreur 403 (edit : précision, avec + de 150 caractères) ! :D

[pierre-24]

Oui mais dans le paramètre ?q=qqch, on peut mettre n'importe quoi, nan?

…

On Feb 5, 2017 3:52 PM, "Situphen" ***@***.***> wrote: Bon, il y a bien le maxlengthqui bloque à 150 caractères ! Si je l'enlève avec la console et que j'envoie le formulaire j'ai une belle erreur 403 ! :D — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub <#3393 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AHhjCw10QJtmeWmCPIQLhqpf4CP0md9gks5rZeIxgaJpZM4HhB4t> .

[gllmc]

En ce qui me concerne, j'ai bien une 403 avec 151 caractères (ou plus), mais pas avec 150.

Exemple avec 151 caractères

[pierre-24]

Donc a priori, c'est bon (151 vs 150, on va pas chipoter, si ?)

…

On Feb 5, 2017 4:19 PM, "Guillaume" ***@***.***> wrote: En ce qui me concerne, j'ai bien une 403 avec 151 caractères (ou plus), mais pas avec 150. Exemple avec 151 caractères <https://beta.zestedesavoir.com/rechercher/?q=https://beta.zestedesavoir.com/rechercher/?q=aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa> — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub <#3393 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AHhjC6mUMPE-z3cJGyryn3vswfnUTv5Xks5rZeiYgaJpZM4HhB4t> .

[artragis]

@vhf tu peux clore du coup?

[pierre-24]

Pas besoin :)

[gllmc]

151 vs 150, on va pas chipoter, si ?

Même pas besoin puisqu'on a le comportement attendu. 150 étant la limite, il est normal qu'on ait une erreur à partir de 151, je me suis mal exprimé. ;)

[Situphen]

Si on pouvait avoir un message au lieu d'une 403 ce serait cool ! Mais bon, c'est déjà pas gênant en l'état actuel car une personne lambda ne touchant pas à la console n'aura jamais cette erreur.