aaaa777 / simple-oauth2

0 stars 0 forks source link

simple-oauth2のハイフンがimportを妨げる #2

Open aaaa777 opened 2 years ago

aaaa777 commented 2 years ago

問題点

リポジトリ名がsimple-oauth2なのでそのままクローンして/CTFd/plugins/simple-oauth2としてしまうとプラグインの外からimport CTFd.plugins.simple-oauth2が出来ない 理由はimportでハイフン(-)が使えないため(pythonでファイル名にハイフンを使うのが非推奨なのを知らなかった)

対策

simple-oauth2依存のプラグインを作らない(import CTFd.plugins.simple_oauth2をしない) どうしてもこのプラグインに手を加えたいならば、flaskのルーティングをオーバーライトするか、forkするべき、という方針 simple-oauth2自体は、プラグイン自身のフォルダ名に依存しないように書いている

インストール方法でフォルダ名を強制させるのも手

aaaa777 commented 2 years ago

プラグインはスタンドアロンで動くように作るべき(APIを設けるのでなければ)と考えているので、import CTFd.plugins.プラグイン名は書くべきではないと思う なので想定したくない