Issue #17 チェックサムファイルの更新スクリプト

Qithub-BOT / QiiCipher

✅ GitHub の SSH 公開鍵でファイルを暗号化およびローカルの秘密鍵で復号・署名・検証するスクリプトのリポジトリです。

Creative Commons Attribution Share Alike 4.0 International

4 stars 3 forks source link

Closed KEINOS closed 3 years ago

KEINOS commented 3 years ago

[x] ハッシュ値の比較スクリプト設置
- bin ディレクトリ内のコマンド・スクリプトのハッシュ値と checksum.sha512 を比較するスクリプト
[x] ハッシュ値の更新スクリプト設置
- bin ディレクトリ内の checksum.sha512 の値を更新するスクリプト
[x] CI に比較スクリプトの実行を追加
[x] 署名ファイルの削除
- 署名ファイル（checksum.sha512.sig）はリリースに含めることにします。リポジトリに入れてもセキュリティの意味はない& PR とは関係がないため。

checksum.sha512 の自動署名と署名ファイル（.sig）の自動化
- これはリポジトリもしくは Organization 用の公開・秘密鍵の設置方法が理解できてから考えます。

KEINOS commented 3 years ago

お手すきでいいので、レビューお願いしまーす。 ping @yoshi389111

yoshi389111 commented 3 years ago

ちょっと気になったのは、スクリプトの一覧があちこちに入ってしまっていることです。

LIST_SCRIPT_BIN="archive check dec enc keygen sign verify checkkeylength dearchive"

今後コマンドを追加する予定はあまりないかと思いますが、もしあった場合修正もれなどがありそうに思います。

例えば、チェックサムファイルを bin ではなく別の場所に配置して、binの中のファイルはすべてコマンドだとした方が良いかもしれない、と思いました。（複数のファイルを直す必要があるので、直すとしても別の PR で直すべきかと思いますが）

この PR としては、問題ないと思います。

LGTM

KEINOS commented 3 years ago

@yoshi389111

ご確認ありがとうございます！

スクリプトの一覧があちこちに入ってしまっている

確かに。漏れますね、ぜったい。とりあえず Issue を別途たてて対応するとして、マージさせていただき 💪 💪 !!