Make ownership filter consistent between relink and rimport

オープン
#32 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
45/100
issue の種類
バグ
明瞭さ
説明が足りない
活発さ
活発
技術スタック
python
領域
authorization, cli

調査の方向性

relink.py の所有者フィルタリングと rimport のディレクトリ走査を比較し、staging の所有者と呼び出し元ユーザーがどのように表現されているかに注目してください。両方のコマンドで所有者を確認すべきなのか、それともどちらも確認すべきでないのかを明確にし、そのうえで、公開と再リンクが別のユーザーのファイルを一貫して扱うことを確認してください。

索引モデルが issue の本文から書いたものです。

説明

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.

主要言語
Python
スター
0
フォーク
2
平均マージ
3日 2時間
マージ済み PR（30日）
2

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

ESMCI/inputdataTools のほかの issue

ESMCI/inputdataTools の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。