Faraday初期化時の動作を変更可能に

aki77 commented 7 years ago

kintoneにproxy経由で接続する必要が出た為に、Faraday初期化時の動作を変更可能にしてみました。もし宜しければ取り込んで頂けると嬉しいです。

coveralls commented 7 years ago

Coverage increased (+0.0008%) to 99.923% when pulling 19b4bd0c19171dc0ee945d7efa7ffb7e58bc159c on aki77:feature-faraday-configurations into 0282d77d94572faa67273782f124712f18f71d86 on jue58:master.

coveralls commented 7 years ago

Coverage increased (+0.0008%) to 99.923% when pulling 19b4bd0c19171dc0ee945d7efa7ffb7e58bc159c on aki77:feature-faraday-configurations into 0282d77d94572faa67273782f124712f18f71d86 on jue58:master.

jue58 commented 6 years ago

@aki77 ありがとうございます！遅くなりました、すいません。

kintoneに直接必要なこと以外(proxyとか！)をブロックで渡して設定するというのはやりたいですが、faradayに直接渡すようにはしたくないですね...

Kintone::Api.new(...)  do |opt|
  opt.proxy = {...}
end

とかで受け付けて、Api#initializeの中でいい感じにしてfaradayに設定するみたいな風にできたら素敵です。とりあえずproxyだけが設定できる形で変更してもらうことはできますか？

jue58 commented 6 years ago

@aki77 あとREADMEにproxyの指定の仕方を追加してもらえるととても助かります。

aki77 commented 6 years ago

faradayに直接渡すようにはしたくないですね

こちらはfaradayに依存したコードにはしたくないという理由からでしょうか？

proxy設定以外でも、kintoneのベーシック認証を利用したいとかhttpリクエストのログを保存したいみたいなケースを想定した場合に、今回の形式の方が利便性が高いと思うのですがいかがでしょうか？例えばoctokitで利用されているsawyerでも同様の形式が取られているようです。

jue58 commented 6 years ago

faradayへは依存しまくってるんでいいんですが、

faradayへの設定に限らずoptionalな設定をするのにブロックを使いたくなるだろう
インターフェースを用意したものだけをつかってもらうような風にしたい

と思ってます。とはいえ、basic認証があるなら(追ってなくてすいません...)使えなくなってしまうこともあると思うので、いったん取り込んでからそのへん変えてみます。

aki77 commented 6 years ago

ありがとうございます！お手数ですがよろしくお願いします。

basic認証については以下のドキュメントにも書いてあるようです。 https://developer.cybozu.io/hc/ja/articles/202531474-kintone-REST-API%E3%81%AE%E8%AA%8D%E8%A8%BC%E8%A8%AD%E5%AE%9A%E3%81%AB%E3%81%A4%E3%81%84%E3%81%A6

jue58 / kintone

Faraday初期化時の動作を変更可能に #11