Make ownership filter consistent between relink and rimport
- 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