ESMCI / ESMCI/inputdataTools

Make ownership filter consistent between relink and rimport

Ouverte
#32 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
Python
Étoiles
0
Forks
2
Merge moyen
3 j 2 h
PR mergées (30 j)
2

Description

As of #31, `rimport` walks directories to find files to publish with no regard for who owns the file. `relink.py`, on the other hand, still filters to just the owner. This is necessary, I think, because the `rimport` makes it so commands are run as the staging owner, and thus we would need special handling to check that ownership by _the calling user_ is checked.

It may be acceptable to go either way on this: Both check ownership or neither does. It feels weird to have this obvious failure case possibly lead to situations where someone publishes someone else's file but then can't relink it, so I lean towards both checking.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Comparez le filtrage de la propriété dans relink.py avec le parcours des répertoires dans rimport, en vous concentrant sur la manière dont sont représentés le propriétaire du staging et l’utilisateur appelant. Déterminez si les deux commandes doivent vérifier la propriété ou si aucune des deux ne doit le faire, puis vérifiez que la publication et la re-liaison traitent de manière cohérente le fichier d’un autre utilisateur.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
authorization, cli
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
Active
Clarté
À clarifier
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.