Faraday: Middleware logger baru saja crash

Ini diharapkan. Tak satu pun dari middleware dapat bekerja tanpa permintaan HTTP dibuat, yaitu tanpa adaptor.

Faraday.new(url: "http://google.com") do |conn|
  conn.response :logger
  conn.adapter Faraday.default_adapter
end

@mislav sementara ini adalah perilaku yang diharapkan, pesan kesalahan yang dihasilkan bisa lebih bermanfaat.

Saya setuju dengan @sethvargo , pesannya bisa lebih baik.

Dalam kasus saya, saya beralih dari tanpa parameter ke Faraday.new dan kemudian menambahkan conn.response :logger . Saya mengharapkan adapter menjadi default_adapter secara default, jadi saya cukup bingung dengan kesalahan baru. Itu tidak membantu bahwa saya sedang men-debug kesalahan lain, jadi saya pikir ini mungkin gejala lain dari kesalahan asli saya.

Saya juga mengalami masalah ini. Saya akan mengatakan bahwa perilaku yang diharapkan adalah Faraday menggunakan adaptor default jika tidak ditentukan.

Jika Anda lebih suka tidak melakukan ini, pengecualian 'adaptor tidak ditentukan' akan menghemat banyak goresan kepala.

@mislav Saya baru saja akan mengatakan apa yang baru saja dikatakan

Oh, saya baru saja digigit oleh hal yang sama. Pesan kesalahan samar untuk kemungkinan kesalahan umum. (Saya menghapus spesifikasi adaptor karena saya keliru menganggapnya tidak perlu eksplisit.)

@mislav Apakah Anda menerima permintaan tarik untuk mengubah kesalahan menjadi sesuatu seperti "Tidak ada adaptor yang ditentukan"?

Atau (tetapi Anda tampaknya tidak menginginkan ini) untuk menganggap "adaptor default" jika tidak ada yang ditentukan secara eksplisit?

Oh, hai @pengwynn , perhatikan bahwa Anda juga bagian dari organisasi ini. Karena Mislav tidak menjawab di atas, ada pemikiran tentang ini? Namun, tidak bermaksud menginjak kaki siapa pun – jika Mislav membuat panggilan ini, saya tidak ingin melewatinya :)

Pesannya harus lebih baik. Meskipun, seperti yang saya katakan, tidak ada middleware yang benar-benar berfungsi tanpa adaptor. Jadi, memperbaiki pesan kesalahan khusus untuk middleware ini meninggalkan kegagalan samar lainnya di middleware lain.

Solusi yang tepat adalah bahwa Faraday membatalkan lebih awal ketika objek Connection tidak menyertakan adaptor di tumpukan. Silakan berlangganan salah satu dari #47, #121, #170

@mislav Aha, terima kasih telah mengklarifikasi!

Saya pikir saya akan mencoba membuat permintaan tarik hanya untuk memperbaiki pesan kesalahan dari logger – sepertinya itu bisa menjadi perubahan yang cukup sederhana yang akan meningkatkan gotcha umum. Semoga tidak apa-apa.

Saya mungkin melihat ke perbaikan yang tepat juga, tetapi tampaknya lebih menantang, jadi saya mungkin tidak punya waktu atau keterampilan untuk mencapainya.