Facteur dans le core en v4.1 ? #5008

Open
opened 11 months ago by cerdic · 13 comments
cerdic commented 11 months ago
Owner

Suite à #4231 et #4957 et à notre politique de release plus fréquente :p
J'ai envoyé la PR spip-contrib-extensions/facteur#12 qui enlève toute référence à un service commercial ou propriétaire

On pourrait donc avantageusement diffuser le plugin dans la distrib core à partir de SPIP 4.1, quitte à le passer en version 5 avec une mise à niveau du code en HP 7.4+ et le nettoyage des trucs obsolètes ?

Suite à #4231 et #4957 et à notre politique de release plus fréquente :p J'ai envoyé la PR https://git.spip.net/spip-contrib-extensions/facteur/pulls/12 qui enlève toute référence à un service commercial ou propriétaire On pourrait donc avantageusement diffuser le plugin dans la distrib core à partir de SPIP 4.1, quitte à le passer en version 5 avec une mise à niveau du code en HP 7.4+ et le nettoyage des trucs obsolètes ?
Owner

J’ai rien contre par principe.

Ceci dit on aimerait aussi sortir une 4.1.0 -alpha très rapidement (on est déjà un peu en retard sur le planning :))

C’est à dire avant ce WE, ou ce WE max.
Je sais pas si c’est jouable ?

J’ai rien contre par principe. Ceci dit on aimerait aussi sortir une 4.1.0 -alpha très rapidement (on est déjà un peu en retard sur le planning :)) C’est à dire avant ce WE, ou ce WE max. Je sais pas si c’est jouable ?
Poster
Owner

vu que Facteur n'est pas neuf, et qu'une alpha c'est pas une béta, rien nous empêche d'envoyer une alpha sans Facteur en annonçant que la beta intègrera le Facteur le temps de nettoyer le code

vu que Facteur n'est pas neuf, et qu'une alpha c'est pas une béta, rien nous empêche d'envoyer une alpha sans Facteur en annonçant que la beta intègrera le Facteur le temps de nettoyer le code
Owner

ou une alpha2 ou alpha3, si vous y tenez.

Une beta, théoriquement, c'est un gel ou on ne fait que corriger des bugs, pas intégrer des nouveautés.

ou une alpha2 ou alpha3, si vous y tenez. Une beta, théoriquement, c'est un gel ou on ne fait que corriger des bugs, pas intégrer des nouveautés.
Owner

Je suis pour sur le principe, mais comme ce plugin utilise la lib phpmailer qui a déjà eu pas mal de releases de sécu, le fait de le coller dans le core impliquera de releaser une version de SPIP dès qu'une faille sera corrigé dans la lib en question. Après, peut-être que la lib est stabilisée maintenant, je n'ai pas souvenir de release de sécu récente.

Je suis pour sur le principe, mais comme ce plugin utilise la lib phpmailer qui a déjà eu pas mal de releases de sécu, le fait de le coller dans le core impliquera de releaser une version de SPIP dès qu'une faille sera corrigé dans la lib en question. Après, peut-être que la lib est stabilisée maintenant, je n'ai pas souvenir de release de sécu récente.
b_b added the
amélioration
label 11 months ago
b_b added this to the 4.1 milestone 11 months ago

Si ça peut aider voilà les mises à jour de sécurité de PHPMailer :
https://github.com/PHPMailer/PHPMailer/blob/master/SECURITY.md

Il y'a eu 6 failles corrigées sur la branche 6.x, cela a donné lieu à 4 mises à jour :
https://github.com/PHPMailer/PHPMailer/releases/tag/v6.5.0 (juin 2021)
https://github.com/PHPMailer/PHPMailer/releases/tag/v6.4.1 (avril 2021)
https://github.com/PHPMailer/PHPMailer/releases/tag/v6.1.6 (mai 2020)
https://github.com/PHPMailer/PHPMailer/releases/tag/v6.0.6 (novembre 2018)

Si ça peut aider voilà les mises à jour de sécurité de PHPMailer : https://github.com/PHPMailer/PHPMailer/blob/master/SECURITY.md Il y'a eu 6 failles corrigées sur la branche 6.x, cela a donné lieu à 4 mises à jour : https://github.com/PHPMailer/PHPMailer/releases/tag/v6.5.0 (juin 2021) https://github.com/PHPMailer/PHPMailer/releases/tag/v6.4.1 (avril 2021) https://github.com/PHPMailer/PHPMailer/releases/tag/v6.1.6 (mai 2020) https://github.com/PHPMailer/PHPMailer/releases/tag/v6.0.6 (novembre 2018)
Poster
Owner

et d'ailleurs on avait une vieille version de PHPMailer dans le plugin ^^ (je viens de mettre à jour sur le master)

et d'ailleurs on avait une vieille version de PHPMailer dans le plugin ^^ (je viens de mettre à jour sur le master)
Owner

Du coup il faut déplacer spip-contrib-extensions/facteur dans spip/facteur ? Ou on le laisserait là bas ?

Du coup il faut déplacer spip-contrib-extensions/facteur dans spip/facteur ? Ou on le laisserait là bas ?
Owner

Notons que dans les plugins spip/* j’avais essayé de rendre les noms de branches d’équerre :

  • les tags ont un 'v'
  • les branches n’ont pas de 'v' et sur 'x.y'

aucazou :)

Notons que dans les plugins spip/* j’avais essayé de rendre les noms de branches d’équerre : - les tags ont un 'v' - les branches n’ont pas de 'v' et sur 'x.y' aucazou :)
Owner

Du coup il faut déplacer spip-contrib-extensions/facteur dans spip/facteur ? Ou on le laisserait là bas ?

Ben je dirais que oui vu que ça correspond à des autorisations de commit différentes.

Sinon, je me posais la question dans l'autre sens pour ceux qu'on a sorti de la dist.

> Du coup il faut déplacer spip-contrib-extensions/facteur dans spip/facteur ? Ou on le laisserait là bas ? Ben je dirais que oui vu que ça correspond à des autorisations de commit différentes. Sinon, je me posais la question dans l'autre sens pour ceux qu'on a sorti de la dist.
Owner

Sinon, je me posais la question dans l'autre sens pour ceux qu'on a sorti de la dist.

Si je ne me trompe pas, on a bien annoncé qu'ils étaient toujours maintenus, et les gens peuvent y proposer des PRs, donc je pense qu'il faut les laisser dans l'orga actuelle :)

> Sinon, je me posais la question dans l'autre sens pour ceux qu'on a sorti de la dist. Si je ne me trompe pas, on a bien annoncé qu'ils étaient toujours maintenus, et les gens peuvent y proposer des PRs, donc je pense qu'il faut les laisser dans l'orga actuelle :)
Owner

Si je ne me trompe pas, on a bien annoncé qu'ils étaient toujours maintenus,

Si je ne me trompe pas on pollue ce ticket :) Je doute qu’on maintienne msie_compat ou même vertèbres encore longtemps… À discuter sur un autre ticket. Mais il faudrait alors modifier leurs descriptions et éventuellement les archiver (lecture seule).

> Si je ne me trompe pas, on a bien annoncé qu'ils étaient toujours maintenus, Si je ne me trompe pas on pollue ce ticket :) Je doute qu’on maintienne msie_compat ou même vertèbres encore longtemps… À discuter sur un autre ticket. Mais il faudrait alors modifier leurs descriptions et éventuellement les archiver (lecture seule).

Si je ne me trompe pas, on a bien annoncé qu'ils étaient toujours maintenus, et les gens peuvent y proposer des PRs, donc je pense qu'il faut les laisser dans l'orga actuelle :)

Je trouve logique qu'on les maintienne en tant que dist tant que ça fait partie de versions de SPIP encore maintenues. Par contre à terme quand ça fait plus partie d'aucune version maintenue… là c'est pas tant les archiver mais oui les déplacer dans la partie "contrib" : ça sera toujours maintenu mais par la communauté, par celleux qui veulent.

> Si je ne me trompe pas, on a bien annoncé qu'ils étaient toujours maintenus, et les gens peuvent y proposer des PRs, donc je pense qu'il faut les laisser dans l'orga actuelle :) Je trouve logique qu'on les maintienne en tant que dist tant que ça fait partie de versions de SPIP encore maintenues. Par contre à terme quand ça fait plus partie d'aucune version maintenue… là c'est pas tant les archiver mais oui les déplacer dans la partie "contrib" : ça sera toujours maintenu mais par la communauté, par celleux qui veulent.
Owner

Je trouve logique qu'on les maintienne en tant que dist tant que ça fait partie de versions de SPIP encore maintenues.

Oui ça parait raisonnable.
Donc ça milite aussi pour que facteur intègre aussi l'orga spip

> Je trouve logique qu'on les maintienne en tant que dist tant que ça fait partie de versions de SPIP encore maintenues. Oui ça parait raisonnable. Donc ça milite aussi pour que facteur intègre aussi l'orga spip
marcimat modified the milestone from 4.1 to 4.2 10 months ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
7 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.