Projet

Général

Profil

Bug #29774

actesweb: mauvais droits sur le fichier .DEM

Ajouté par Serghei Mihai (congés, retour 15/05) il y a plus de 5 ans. Mis à jour il y a environ 5 ans.

Statut:
Fermé
Priorité:
Normal
Version cible:
-
Début:
15 janvier 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:

Description

Le fichier .DEM créé à coup de tempfile.NamedTemporaryFile appartient strictement à son user: 500.

De ce fait le compte utilisé par le client pour se connecter en SFTP n'a pas les droits de venir lire/déplacer ce fichier.

Révisions associées

Révision 1a56a4c2 (diff)
Ajouté par Serghei Mihai (congés, retour 15/05) il y a plus de 5 ans

actesweb: fix demand file permissions (#29774)

Historique

#2

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 5 ans

  • Statut changé de Nouveau à En cours

Branche wip/29774-actesweb-fix-temporary-file-permissions.

os.rename(tpf.name, filepath) sorti du contexte manager car le fichier n'est pas encore fermé au moment du renommage.

#3

Mis à jour par Thomas Noël il y a plus de 5 ans

Serghei Mihai a écrit :

Branche wip/29774-actesweb-fix-temporary-file-permissions.

stat.S_IRUSR|stat.S_IWUSR|stat.S_IRGRP|stat.S_IWGRP ça me semble beaucoup de W. Si le fichier n'est pas destiné à être modifié (ce que je crois), je vote pour stat.S_IRUSR|stat.S_IRGRP only.

#4

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 5 ans

Yep, tu as raison.
Branche à jour.

#5

Mis à jour par Thomas Noël il y a plus de 5 ans

  • Statut changé de En cours à Solution validée

Détail, "group car" -> "group can" dans les tests.

#6

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 5 ans

Corrigé.

#7

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 5 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 1a56a4c21ea8d2d7db7838e67fc4867fb6366812 (origin/master, origin/HEAD)
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Tue Jan 15 16:05:33 2019 +0100

    actesweb: fix demand file permissions (#29774)
#8

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#9

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

  • Statut changé de Solution déployée à Fermé

Formats disponibles : Atom PDF