Development #68052
ne pas inclure "tryauth" pour les visites par des robots
0%
Description
Ils ne seront de toute façon pas authentifiés mais ça leur donne l'occasion de se perdre sur l'authentification, parce que même si le IsPassive=true y assure une redirection automatique ça fait un enchainement ou un changement de domaine mal géré.
(insérer sarcasme sur google)
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a plus d'un an
- Fichier 0001-wcs-do-not-add-tryauth-for-bots-68052.patch 0001-wcs-do-not-add-tryauth-for-bots-68052.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus d'un an
Il y a aussi tryauth dans combo/apps/wcs/templates/combo/wcs/form_link_fragment.html.
Mis à jour par Thomas Noël il y a plus d'un an
- Fichier 0001-wcs-do-not-add-tryauth-for-bots-68052.patch 0001-wcs-do-not-add-tryauth-for-bots-68052.patch ajouté
Et c'était sans doute le plus important.
Mis à jour par Frédéric Péters il y a plus d'un an
J'éliminerais aussi le ?cancelurl en cas de bot.
Aussi je trouve la forme actuelle peu lisible; je préférerais if not is_a_bot(...)
.
Les modifications aux {% include ... %} ne me semblent pas nécessaires, on ne passe pas "only", les variables du contexte doivent donc aussi être disponibles dans le gabarit inclus.
Mis à jour par Thomas Noël il y a plus d'un an
- Fichier 0001-wcs-do-not-add-tryauth-for-bots-68052.patch 0001-wcs-do-not-add-tryauth-for-bots-68052.patch ajouté
Voici, sans cancelurl et avec un "is_a_bot" (très rudimentaire mais moi ça me suffit).
Mis à jour par Frédéric Péters il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
'googlebot' in (request.headers.get('User-Agent')
Tu peux juste 'bot' in ...
? (i.e. pas spécifique google).
Mis à jour par Thomas Noël il y a plus d'un an
Frédéric Péters a écrit :
Tu peux juste
'bot' in ...
? (i.e. pas spécifique google).
Je m'inquiétais d'un navigateur qui aurait un jour ces trois lettres dans son User-Agent. Mais je viens de lire https://en.wikipedia.org/wiki/User_agent et : « by convention the word "bot" is included in the name of the agent »
C'est donc parti avec "bot".
(Mais jenkin rouge.)
Mis à jour par Thomas Noël il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
Et Jenkins était rouge pour tout autre chose
commit 12087eb075126c3e2d6ed6e6fe20814b3e7058c0 Author: Thomas NOËL <tnoel@entrouvert.com> Date: Tue Aug 9 12:18:23 2022 +0200 wcs: do not add tryauth for bots (#68052)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
wcs: do not add tryauth for bots (#68052)