Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

review-initへのアーカイブロード展開の機能の追加 #1105

Merged
merged 2 commits into from
Oct 15, 2018

Conversation

kmuto
Copy link
Owner

@kmuto kmuto commented Sep 1, 2018

#1102 の対応。
プロジェクトフォルダの作成後にzipアーカイブを単純に上書き展開するという仕掛けです。

  • -p ローカルファイルパス または -p https://〜-p http://〜 でzipファイルを指定します。
  • ネットワークダウンロードは Net::HTTP なので301などには対応していません。パスワード制限のところにも対応していません。まぁこのあたりはダウンロードしてから指定してもらうのがよさそう。open-uriは今は使うべきではない、になっているんですね…
  • 一応「/」から始まるパス、および「/../」が含まれるパスは発見次第展開を中止するという安全策は入れています。
  • パスの日本語まわりなどはOSのロケール状態まで見るのもきついので対処していません。

This was referenced Sep 3, 2018
@takahashim
Copy link
Collaborator

ちょっとTempfileの扱いが気になりますが、挙動自体は問題ないかと思うので、これでマージしますね

@takahashim takahashim merged commit 915a811 into master Oct 15, 2018
@kmuto kmuto deleted the init-from-net branch December 19, 2019 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants