ishkawa/APIKit

RequestType#configureURLRequest

minerva1129 opened this issue · 3 comments

minerva1129 commented

日本語ですみません。
当方swiftに関しては使い始めたばかりで見当違いな指摘かもしれませんが、RequestTypeのconfigureURLRequestの戻り値が使われていないような気がします。副作用でのconfigurationを期待するメソッドなのでしょうか？

ishkawa commented

戻り値が使われるのが正しい動作ですね。
これまで副作用だけを利用していて見落としていました！

ishkawa commented

I've fixed this issue, the return value of configureURLRequest(_:) is not used in buildURLRequest(). This will be released in upcoming version 1.4.1, and also be picked on 2.0 branch.

Thank you for the report ✨

  • ❤️1
minerva1129 commented

素早い対応ありがとうございました！！🙇🏻