3liz / QgisCadastrePlugin

A QGIS plugin which helps users to import the french land registry ('cadastre') data into a database. It is meant to ease the use of the data in QGIS by providing search tools and appropriate layer symbology.
GNU General Public License v2.0
61 stars 41 forks source link

Fix tarfile security issue #457

Closed Gustry closed 1 month ago

Gustry commented 1 month ago

Superseeds #434 for all Python version

Gustry commented 1 month ago

Thanks for the review @dmarteau

OK for you ?

landryb commented 1 month ago

J'aurais peut-être mis un message plus explicite en justifiant la mise à jour par l'existence d'une faille de sécurité, comme ça les utilisateurs sont plus enclins à upgrader.

je ne sais pas bien comment ca fonctionne sous windows, mais il me semble que les utilisateurs n'ont pas forcément le choix de la version de python bundlée avec leur installation de QGIS .. non ? et ils ne savent probablement pas ce qu'est python non plus :)

Gustry commented 1 month ago

Oui, entièrement d'accord. C'est pour ca que je ne voulais pas passer du temps de dessus, voir la discussion dans le PR précédent #434 ;-)

Quasi 50% des utilisateurs de Cadastre sont en Python 3.9.5 :)