fix: eviter un risque d'incoherence entre le nettoyage des variables dans le contexte des cache et le nettoyage des urls self() utilisée par la pagination #5765

Merged
marcimat merged 4 commits from fix_unifie_contexte_ignore_variables into master 2 weeks ago
cerdic commented 3 weeks ago
Owner

notamment en cas de personalisation de la constante

Du coup j'ai renommé la constante qui devient un tableau de variables exprimées en pseudo bout de preg

Refs: spip-team/securite#4848

notamment en cas de personalisation de la constante Du coup j'ai renommé la constante qui devient un tableau de variables exprimées en pseudo bout de preg Refs: https://git.spip.net/spip-team/securite/issues/4848
cerdic added 1 commit 3 weeks ago
b_b approved these changes 3 weeks ago
Owner

Si je suis bien

Ça introduit:

  • _CONTEXTE_IGNORE_LISTE_VARIABLES

Ça supprime:

  • _CONTEXTE_IGNORE_VARIABLES

Est-ce qu’il faut reporter en 4.2 ?

Si je suis bien Ça introduit: - `_CONTEXTE_IGNORE_LISTE_VARIABLES` Ça supprime: - `_CONTEXTE_IGNORE_VARIABLES` Est-ce qu’il faut reporter en 4.2 ?
cerdic commented 2 weeks ago
Poster
Owner

je suis pour pas reporter en 4.2 : c'est une amélioration pour les rares cas où l'on personalise la constante, et du coup elle est renomée et change de format, donc ça introduit une incompat

je suis pour pas reporter en 4.2 : c'est une amélioration pour les rares cas où l'on personalise la constante, et du coup elle est renomée et change de format, donc ça introduit une incompat
marcimat referenced this issue from a commit 2 weeks ago
marcimat force-pushed fix_unifie_contexte_ignore_variables from 5385bee4b8 to 7569f90cf1 2 weeks ago
erational approved these changes 2 weeks ago
b_b approved these changes 2 weeks ago
marcimat merged commit 7569f90cf1 into master 2 weeks ago
marcimat deleted branch fix_unifie_contexte_ignore_variables 2 weeks ago

Reviewers

erational approved these changes 2 weeks ago
b_b approved these changes 2 weeks ago
The pull request has been merged as 7569f90cf1.
Sign in to join this conversation.
Loading…
There is no content yet.