Faraday: Net :: Adaptor HTTP mencoba ulang permintaan secara otomatis

Dibuat pada 21 Feb 2018  ·  3Komentar  ·  Sumber: lostisland/faraday

Info dasar

  • Versi Faraday: apa saja
  • Versi Ruby: 2.5+

Deskripsi masalah

Saat menggunakan Net::HTTP sebagai adaptor, pengguna akan terkejut dengan strategi coba lagi yang disediakan oleh Net::HTTP . Ini membawa ke "masalah" yang berbeda (misalnya #612 dan #771).
Tidak banyak yang dapat kami lakukan untuk Ruby <= 2.4, namun Ruby 2.5 menyediakan konfigurasi max_retries untuk Net::HTTP sehingga Faraday harus mengaturnya secara otomatis jika tersedia.

Langkah-langkah untuk mereproduksi

Lihat masalah #612 dan #771.

feature help wanted

Semua 3 komentar

Saya telah melihat masalah ini. Salah satu pendekatan yang disebutkan di #771 adalah menyetel default ke 0, tetapi mengubahnya tergantung pada jumlah percobaan ulang yang dikonfigurasi. Kami harus membuat middleware Coba Lagi mengetahui adaptor yang mendasarinya (karena beberapa adaptor mungkin tidak mendukung ini), dan menerapkan logika ini secara kondisional di env . Adaptor juga harus mengetahui jumlah percobaan ulang.

Sebagai alternatif dari pendekatan tersebut, kita dapat mengaturnya agar tidak melakukan percobaan ulang, alih-alih menjaga logika percobaan ulang terbatas pada middleware Coba lagi. Apa pun hasilnya, saya akan dengan senang hati mengambilnya @iMacTia .

Saya benar-benar memikirkan opsi kedua.
Kami telah memiliki middleware Coba lagi yang dapat menangani permintaan yang gagal dicoba kembali.
Adaptor (dalam hal ini Net::HTTP) tidak boleh mencoba kembali permintaan sendiri.

Yang perlu kita lakukan dalam kasus ini adalah memberi tahu Net::HTTP adapter untuk mengatur max_retries ke 0, jika properti ini tersedia, tentu saja

@iMacTia - Saya sangat setuju. Dalam hal ini, saya akan mengambil ini.

Apakah halaman ini membantu?
0 / 5 - 0 peringkat

Masalah terkait

yusefu picture yusefu  ·  3Komentar

jedeleh picture jedeleh  ·  3Komentar

QuinnWilton picture QuinnWilton  ·  4Komentar

mvastola picture mvastola  ·  4Komentar

ioquatix picture ioquatix  ·  4Komentar