Faraday: निकायों के साथ हटाता है

आपकी समीक्षा के लिए शीघ्र ही एक पीआर तैयार किया जाएगा।

हाय @ कार्लबीसी18 ,

क्या आप कृपया उपरोक्त परिवर्तनों के लिए एक लिंक प्रदान कर सकते हैं?
मुझे लगता है कि यह केवल HTTP 2.0 विनिर्देशों के लिए सही है, इसलिए हमें इस सुविधा पर ध्यान से विचार करने की आवश्यकता हो सकती है।

https://tools.ietf.org/html/rfc7231#section -4.3.5

FYI करें: टेक्स्ट का यही ब्लॉक GET && HEAD अनुरोधों पर भी लागू होता है।

विशेष रूप से कल्पना का यह हिस्सा:

DELETE अनुरोध संदेश के भीतर एक पेलोड में कोई परिभाषित शब्दार्थ नहीं होता है;
DELETE अनुरोध पर पेलोड बॉडी भेजने से कुछ मौजूदा हो सकता है
अनुरोध को अस्वीकार करने के लिए कार्यान्वयन।

मेरे पास परीक्षणों के साथ एक कोड परिवर्तन है जो काम करता है (सुनिश्चित नहीं है कि आप इसे चाहते हैं)। मैं इस परियोजना के लिए जनसंपर्क कैसे बनाऊं?

मुझे लगता है कि यह केवल HTTP 2.0 विनिर्देशों के लिए सही है, इसलिए हमें इस सुविधा पर ध्यान से विचार करने की आवश्यकता हो सकती है।

यह वास्तव में HTTP 1.1 का हिस्सा है

यदि आप पीआर बनाना चाहते हैं तो मैं निश्चित रूप से इस पर विचार करूंगा।
आप इसे किसी भी अन्य रेपो के साथ कर सकते हैं, बस faraday फोर्क करें और अपने फोर्क पर परिवर्तन लागू करें, फिर हमारी मास्टर शाखा में पीआर बनाएं।

धन्यवाद @iMacTia। मैं इसे लगाऊंगा। संस्करण को बदलने के बारे में आपके क्या विचार हैं, अंत में इसके साथ एक प्रमुख संस्करण संख्या है? यह बदलाव कई लोगों के लिए टूट सकता है। मैं इस बिंदु पर सेवर पैटर्न का पालन करना चाहता हूं, और संस्करण फैराडे 1.0.0 होना चाहता हूं। मुझे अपने विचारों को जानने दें।

हम निश्चित रूप से सेवर से चिपके रहने की कोशिश कर रहे हैं और हमने कुछ महत्वपूर्ण बदलावों के साथ v1.0 शेड्यूल किया है।
यदि यह परिवर्तन पश्चगामी संगतता को तोड़ने वाला है, तो इसे केवल संस्करण 1.0 में जारी किया जाएगा (जिसकी वर्तमान में कोई निश्चित रिलीज़ तिथि नहीं है)।
हालांकि, मेरी समझ यह है कि अब आप वैकल्पिक रूप से GET, HEAD और DELETE अनुरोधों में एक बॉडी जोड़ सकते हैं, तो क्या यह पिछड़ा संगत नहीं होना चाहिए? क्या मैं कुछ भूल रहा हूँ?

@iMacTia मैं

यह मेरे https://github.com/lostisland/faraday/pull/695#issuecomment -305145499 पर अनुवर्ती कार्रवाई है।
सबसे पहले हमें यह तय करना चाहिए कि हम इसे कैसे काम करना चाहेंगे।
यह मानते हुए कि परिवर्तन पिछड़ा संगत होना चाहिए, नीचे अभी भी काम करना चाहिए:

conn = Faraday.new(...)
conn.get('/path', {page: 1, per: 10})

# performs "GET /path?page=1&per=10"

विकल्प 1 जैसा कि मेरी टिप्पणी में कहा गया है, इस व्यवहार को विन्यास योग्य बनाना होगा:

conn = Faraday.new(..., standard: :http1_1)
conn.get('/path', {page: 1, per: 10})

# performs "GET /path" with body "page=1&per=10" (assuming www_form_url_encoded request)

इसके बजाय विकल्प 2 का अर्थ कुछ अधिक शक्तिशाली होगा:

conn = Faraday.new(...) # note no special config here
conn.get('/path', {page: 1, per: 10}, body: {some_key: 'some_value')

# performs "GET /path?page=1&per=10" with body "some_key=some_value"

हालांकि एक समस्या है, विकल्प 2 केवल रूबी> = 2 के साथ काम कर सकता है क्योंकि रूबी 1.9 कीवर्ड पैरा का समर्थन नहीं करता है।

मुझे लगता है कि इस संबंध में रूबी 1.9 पर काम नहीं कर रहा है, मुझे लगता है कि यह ठीक है। रूबी 1.9 कई साल पुरानी है। मैं न्यूनतम रूबी संस्करण के लिए संस्करण को कम से कम 2.0 तक टक्कर देने की वकालत करता हूं, जो फैराडे के v1.0 रिलीज के साथ इनलाइन जाएगा। इसके अलावा विकल्प 2 मेरी राय में http 1.1 spec का अधिक बारीकी से पालन करता है क्योंकि आपके पास पैरा और या वैकल्पिक निकाय पास करने का विकल्प होगा।

मुझे लगता है कि रूबी 1.9 के साथ काम करने वाली http कल्पना सुविधा के साथ पिछड़ा संगतता बनाए रखने का प्रयास करने से समस्याएं पैदा हो रही हैं। मुझे लगता है कि यह सुविधा फैराडे के V1 के लिए बेहतर अनुकूल होगी। मुझे ऐसा लगता है कि इस परिवर्तन के समर्थन के लिए बहुत सारे चर की आवश्यकता थी (रूबी वी, पैरा पासिंग)। शायद मैं ज्यादा सोच रहा हूँ....

मुझे लगता है कि इस संबंध में रूबी 1.9 पर काम नहीं कर रहा है, मुझे लगता है कि यह ठीक है। रूबी 1.9 कई साल पुरानी है। मैं न्यूनतम रूबी संस्करण के लिए संस्करण को कम से कम 2.0 तक टक्कर देने की वकालत करता हूं, जो फैराडे के v1.0 रिलीज के साथ इनलाइन जाएगा। इसके अलावा विकल्प 2 मेरी राय में http 1.1 spec का अधिक बारीकी से पालन करता है क्योंकि आपके पास पैरा और या वैकल्पिक निकाय पास करने का विकल्प होगा।

मैं उपरोक्त सभी से पूरी तरह सहमत हूं। यह वास्तव में कार्रवाई का सबसे अच्छा तरीका है, भले ही इसका मतलब है कि हमें अभी भी v1.0 . की प्रतीक्षा करनी होगी

तो एक v1.0 पिछड़े संगत एपीआई को परिभाषित करने के लिए, हम निम्नलिखित प्रयास कर सकते हैं:

प्राप्त करें, सिर, अनुरोध हटाएं

conn = Faraday.new(...) # note no special config here
conn.get(path, url_params, body: {...})
# second parameter is URL PARAMS, optionally accept a named parameter for request body
# works the same for head and delete

पोस्ट, पुट, पैच अनुरोध

conn = Faraday.new(...) # note no special config here
conn.post(path, body, url_params: {...})
# second parameter is REQUEST BODY, optionally accept a named parameter for url_params
# works the same for put and patch

मुझे लगता है कि इस तरह हम हर संभव संयोजन को कवर कर रहे हैं और पोस्ट/पुट/पैच अनुरोधों को भी बढ़ा रहे हैं!

निश्चित रूप से सहमत हैं। क्या हम अपने पीआर को अस्वीकार करना चाहते हैं और इसके लिए एक नया खोलना चाहते हैं? क्या कोई V1 शाखा कट है जिससे हम काम कर सकते हैं?

उसे बंद करने और खरोंच से शुरू करने में खुशी हुई क्योंकि यह पूरी तरह से अलग होगा। उसके लिए खेद है!

मैं विलय करने से पहले एक वी 1 शाखा बनाउंगा ताकि मास्टर से शुरू करने के लिए स्वतंत्र महसूस हो और मैं पीआर को रीडायरेक्ट कर दूंगा :)

नमस्ते @ Carlbc18 बस आपको यह बताना चाहता था कि शाखा 1.0 अब आपके लिए उपलब्ध है।

शुक्रिया शुक्रिया! मैं

अभी भी इस पर प्रतीक्षा कर रहे किसी के लिए, मुझे निम्नलिखित समाधान मिला जो मेरे मामले में काम करता प्रतीत होता है:

conn = Faraday.new
body = {
some_key: 'some value'
}
conn.run_request(:delete, url, body, headers)

मुझे आशा है कि यह किसी को तब तक मदद करता है जब तक कि डिफ़ॉल्ट रूप से शरीर को हटा नहीं देता

तो क्या यह उपलब्ध होने पर कोई अपडेट है?

मैंने अभी तक कोई जनसंपर्क नहीं देखा है, लेकिन v1 पर काम धीरे-धीरे आगे बढ़ रहा है।
मैं खुद इस पर काम करने में सक्षम हो सकता हूं लेकिन मैं इस समय कोई अनुमान नहीं दे सकता।
अनुरोधों को कैसे काम करना चाहिए, इस पर अद्यतन एपीआई को ऊपर मेरी टिप्पणी पर समझाया गया है, इसलिए यदि कोई मुझसे पहले इस पर एक शॉट लेना चाहता है, तो एक पीआर का बहुत स्वागत है।

पूरी तरह से पिछड़े संगत समाधान के लिए, पीआर को मास्टर से बाहर किया जा सकता है, और मैं एक और 0.x रिलीज कर सकता हूं।
यदि परिवर्तन पश्चगामी संगतता को तोड़ने वाला है तो 1.0 शाखा से बाहर शाखा करना बेहतर है

https://github.com/lostisland/faraday/pull/855 इसका समर्थन करने के लिए एक मसौदा पीआर है।

मैं इस सुविधा को अस्वीकार कर रहा हूं. आप एक निकाय के साथ अनुरोध करने के लिए फैराडे में ब्लॉक फॉर्म का उपयोग कर सकते हैं:

conn.delete(url) do |req|
  req.body = { some_key: 'some value' }
end

इस विचार ने निकायों के साथ एक सिंगल लाइनर को सरल हटा दिया होगा, लेकिन कार्यान्वयन मेरी इच्छा से अधिक जटिलता और आश्चर्यजनक व्यवहार जोड़ता है। यह ब्लॉक फॉर्म फैराडे में पहले से ही बहुत अच्छा काम करता है। मैं किसी विधि में अधिक तर्क जोड़ने के लिए कक्षा में गुणों को संशोधित करना पसंद करता हूं:

conn.delete('/foo/1') do |req|
  req.headers["X-Men"] = "marvel"
  req.params[:confirm] = 1
  req.body = { some_key: 'some value' }
end