vital.vim icon indicating copy to clipboard operation
vital.vim copied to clipboard

RFC: Web.HTTP の URL 関係を Web.URI に移動してほしい

Open lambdalisue opened this issue 8 years ago • 8 comments

具体的には https://github.com/vim-jp/vital.vim/blob/master/doc/vital-web-http.txt#L113-L120 の辺

Web.HTTP の書き直し(or Network.HTTP の作成)で利用したいので URL 関係の処理は別モジュールにしたいと思うのですがどうでしょうか?

関連 https://github.com/vim-jp/vital.vim/issues/361 https://github.com/vim-jp/vital.vim/issues/373 https://github.com/vim-jp/vital.vim/issues/357 https://github.com/vim-jp/vital.vim/pull/372

lambdalisue avatar Jan 18 '16 02:01 lambdalisue

URI.decode()URI.encode() はすでにあります。 application/x-www-form-urlencoded については、URI の扱う範疇を超えているので、ちょっと考える必要があります。

thinca avatar Jan 18 '16 02:01 thinca

URI の扱う範疇を超えているので、ちょっと考える必要があります。

問題はそこなのですが Web.HTTP はリクエストだけを扱うほうが楽なんですよね。テストとかの関係上

lambdalisue avatar Jan 18 '16 02:01 lambdalisue

Web.HTTP は廃棄の方向で再実装でもいい気もしてます。(どちらにしても URI のやつと微妙に動作がかぶるので利用者にわかりづらいというのはある)

thinca avatar Jan 18 '16 02:01 thinca

Web.HTTP は廃棄の方向で再実装でもいい気もしてます。

はい、基本そのつもりでしたが実装重いので Web.HTTP から使えるものをほかモジュールに移行しつつ Network.HTTP の作成に着手したいと思い RFC しています。Network.HTTP はリクエストだけを扱いたい(URI 関係のテストは別に投げたい)

lambdalisue avatar Jan 18 '16 02:01 lambdalisue

@tyru さんどう思います? (Web.URI 書いてる関係上)

そろそろ System.Process が固まりそうなので、こちらも固めたい。 個人的には encodeURIComponentWeb.URI にあったほうが楽ですが、URIの範疇超えてるし Network.HTTP でやるべきというならばがんばります。

c.f. @thinca

lambdalisue avatar Mar 17 '16 11:03 lambdalisue

急いで入れて Changes の行を増やすのもアレなので、とりあえず今はサポート無しで良いんじゃないですかね。 ありすえさんが application/x-www-form-urlencoded の実装が今すぐ欲しいなら別ですが。

tyru avatar Mar 17 '16 12:03 tyru

急いでという意味ではありませんでした。実装を行うというよりかは最終形がどちらになるのかを固めたかったという感じです

lambdalisue avatar Mar 17 '16 12:03 lambdalisue

了解です。 自分としては、@thincaさんも言ってる通りどこにどのメソッドを入れるか決めあぐねている状態なら、今は入れなくていいんじゃないかな、程度でした。

application/x-www-form-urlencoded については、URI の扱う範疇を超えているので、ちょっと考える必要があります。

tyru avatar Mar 17 '16 15:03 tyru