yue4u / dhu Goto Github PK

現状、dhu loginコマンドはidとパスワードを保存するだけで「正しい情報か」を確認しません。
これだと間違えた時に気付きづらいので、情報を入力した後にログインできるかを試行したいです
具体的には、login() を試行し、失敗したら再度入力を受け付ける(もしくは間違っていることを表示して終了する)等したいです

@okatechnology 先生が実装してくれそうなのでissueを立てます

https://github.com/rainy-me/dhu/blob/8deb30e2b1bc71d0973d9f93c66c3ab96fccaa34/packages/cli/index.ts#L53-L61

ここにsortのオプションを追加して、

https://github.com/rainy-me/dhu/blob/8deb30e2b1bc71d0973d9f93c66c3ab96fccaa34/packages/cli/view/task.ts#L17

この関数に渡して、実装すればできます。

データ構造的にソートしずらいかもしれないが、判断お任せます。

もしかすると、ここで 2020/05/12(火) 00:00 みたいな文字列を一般的なタイムフォーマットにノーマライズしたほうがソートしやすいかもしれません。

https://github.com/rainy-me/dhu/blob/8deb30e2b1bc71d0973d9f93c66c3ab96fccaa34/packages/core/task.ts#L80-L81

現在デジタルハリウッド大学2年生の者です。現在このリポジトリでメンテナーを募集しているとのことでしたのでメンテナーになりたいです。

パスワードを打つ際、画面に表示されるのはセキュリティ上あまり良くないので、なんとか隠したいです。
ですが、軽く調べた所nodeのreadlineでは標準でその機能はないようですね...
npm/readは求めている機能を持っているようですが、最終更新が6年前なので使えるのか微妙です...(そもそもTypeScriptと上手く動くのかもわかりません)

調べると方法はあるみたいですが、どれが良いのかとかは良く分からないです
https://stackoverflow.com/questions/24037545/how-to-hide-password-in-the-nodejs-console

https://github.com/semantic-release/semantic-release

1. It's too hacky for trying to fix this right now. see gridsome/gridsome#551
2. gridsome itself is not under active development, I'm considering move to other SSG
3. I won't be actively maintenance this site in predictable future due to lack of interest

現状、loginに失敗すると以下のようにUnhandledPromiseRejectionが吐かれます。

ちょっとエラーとして見辛いので、handleしたいなと思っています。

ちなみに自分でもPromiseのthenを使って一部だけ書いてみたのですが、

以下のようにエラーを表示した後固まってしまい、どうすれば良いのか分からない状況です...

現状、ログイン情報は平文で保存してると思いますが、出来ればpass等を使って暗号化して保存したいです。
ただpassを使っているユーザーも多くはないと思うので、オプションでpassを使うように出来ると良いなと思っています。
(正直ニーズが個人の趣味レベルなので、需要がなさそうだなと判断されたら自分のforkでのみ作ろうと思っています)

see README.