Apollo-link-rest: 404 त्रुटियों से निपटने के लिए अनुशंसित रणनीति?

को निर्मित 6 जून 2018 · 12टिप्पणियाँ · स्रोत: apollographql/apollo-link-rest

एक आरईएसटी एपीआई के लिए यह इंगित करने के लिए 404 प्रतिक्रिया का उपयोग करना काफी आम है कि कोई रिकॉर्ड मौजूद नहीं है। अभी जब ऐसा होता है तो परिणाम networkError होता है, जो कि डिफ़ॉल्ट रूप से, अपोलो स्टैक द्वारा ग्राफ़कॉल त्रुटि की तुलना में अधिक घातक त्रुटि के रूप में माना जाता है।

उदाहरण के लिए, क्वेरी के साथ:

query BookQuery($slug: ID!) {
  book(slug: $slug) @rest(type: "Book", path: "book/:slug") {
    name
    author
  }
}

यदि एंडपॉइंट ४०४ लौटाता है तो यह एक नेटवर्क त्रुटि के साथ अस्वीकार करता है जिसे संभाला जाना चाहिए जहां इरादा केवल error रेंडर प्रोप का उपयोग उपयोगकर्ता को गैर-घातक त्रुटि संदेश प्रदर्शित करने के लिए करता है। यानी अगर यह REST के बजाय एक GraphQL क्वेरी थी तो इसे नेटवर्क त्रुटि के रूप में वर्गीकृत नहीं किया जाएगा और इसे इस तरह से संभाला जाएगा।

मैंने इसे ठीक करने के लिए अपने कस्टम error-link एक चेक जोड़ना था जो 404 नेटवर्क त्रुटि को GraphQL त्रुटि में बदल देता है:

forward(operation).subscribe({
  next: result => {...},
  error: networkError => {
    if (networkError.statusCode === 404) {
      return observer.next({errors: [networkError]});
    }
    //...
    observer.error(networkError);
  },
  complete: observer.complete.bind(observer),
});

इसमें अभी भी सुधार की गुंजाइश है, लेकिन यह समस्या का समाधान करता है।

मैं सोच रहा था कि क्या इसे किसी तरह apollo-link-rest में संबोधित किया जाना चाहिए, इसलिए मुझे लगा कि मैं बातचीत शुरू करूंगा।

enhancement💡 question❔

स्रोत

marnusw

👍3

सभी 12 टिप्पणियाँ

यह एक अच्छा सवाल है। - मैं इस पैटर्न को दस्तावेज़ीकरण के सुझाव के रूप में जोड़ा जाना पसंद करूंगा। मुझे लगता है कि यह समस्याओं का एक वर्ग है जिससे कई अपोलो-लिंक पीड़ित हैं (अपोलो-लिंक-स्टेट दिमाग में आता है)।

मैं नहीं जानता कि हमें यकीन है कि यह अपोलोलिंक के सभी उपयोगकर्ताओं के लिए सही उत्तर है, हालांकि?

fbartho 7 जून 2018

😕1 👎1

इस समय सभी http त्रुटियों को अपोलो-लिंक-रेस्ट में नियंत्रित किया जाता है

        if (res.status >= 300) {
          // Throw a JSError, that will be available under the
          // "Network error" category in apollo-link-error
          let parsed: any;
          try {
            parsed = await res.json();
          } catch (error) {
            // its not json
            parsed = await res.text();
          }
          rethrowServerSideError(
            res,
            parsed,
            `Response not successful: Received status code ${res.status}`,
          );
        }
        return res;

शायद हम एक केस मिलान फ़ंक्शन जोड़ते हैं जो उपयोगकर्ताओं को यह निर्धारित करने देता है कि त्रुटि कैसे संसाधित की जाती है?
404 == null जैसे सामान्य मामलों के लिए कुछ उपयोगी डिफ़ॉल्ट उपयोगिताएँ प्रदान कर सकता है

paulpdaniels 11 जून 2018

जबकि मैं @fbartho की चिंता साझा करता

query postAndTags {
  post @rest(type: "Post", path: "/post/1") {
    id
    title
  }
  tags @rest(type: "[Tag]", path: "/tags") {
    name
  }
}

और post मौजूद नहीं है और 404 लौटाता है तो पूरी क्वेरी नेटवर्क त्रुटि पर विफल हो जाती है। हालांकि यह संभवतः कुछ मामलों में वांछित हो सकता है, ऐसा लगता है कि परिणाम होना चाहिए:

const data = {
  post: null,
  tags: [{ name: 'apollo' }, { name: 'graphql' }],
}

अगर हम इसे पूरा करना चाहते हैं, तो @paulpdaniels के सुझाव के अनुसार अतिरिक्त त्रुटि प्रबंधन जोड़ना बहुत आसान होगा। 4xx त्रुटियों को देखते हुए मैं वास्तव में केवल 404 और 400/412/422 (म्यूटेशन पर) देख सकता हूं, संभवतः गैर-घातक त्रुटियां हैं जिन्हें विशेष उपचार की आवश्यकता होती है, या तो ग्राफ़क्यूएल त्रुटियों या परिणाम को null ।

सवाल यह है कि क्या इन त्रुटियों के लिए कुछ निश्चित हैंडलर प्रदान करना उचित है या यदि इसे error-link या आवेदन पर छोड़ दिया जाना चाहिए। चूंकि एकाधिक उत्परिवर्तन प्रश्नों का समर्थन नहीं किया जाता है (मुझे लगता है?) केवल वर्तमान शॉर्ट आ रहा है जो विशेष रूप से प्रासंगिक लगता है वह 404 उदाहरण है जिसे मैंने ऊपर से शुरू किया था।

कॉन्फ़िगरेशन में एक त्रुटि हैंडलर जोड़ने का विकल्प भी है, लेकिन ऐसा लगता है कि यह कार्यक्षमता को error-link से दूर कर सकता है जो वास्तव में वहां से संबंधित है। यह इस लिंक के एपीआई को अनावश्यक रूप से जटिल भी बना सकता है।

marnusw 12 जून 2018

👍1

@marnusw यह एक अच्छी प्रतिक्रिया है। मैं एक सुधार का समर्थन करूंगा जो आपके व्यवहार को डिफ़ॉल्ट रूप से लागू करता है!

fbartho 21 अग॰ 2018

142 404 त्रुटि प्रबंधन जोड़ता है। मुझे लगता है कि इस चर्चा से यह सबसे महत्वपूर्ण बात है।

marnusw 1 सित॰ 2018

👎2

कई बाकी एपीआई मानकों के आधार पर, सभी त्रुटि कोड में हमेशा एक मानव-पठनीय संदेश शामिल होना चाहिए। उस परिणाम को रद्द करना, जैसा कि #142 में किया गया था, उसके सीधे विरोध में है।

christrude 13 अप्रैल 2021

मैंने ४०४ को सामान्य नेटवर्क त्रुटियों के रूप में बहाल करने के लिए एक पीआर रखा है, अच्छे आरईएसटी एपीआई प्रथाओं के साथ मेल खाने के लिए: https://github.com/apollographql/apolo-link-rest/pull/283

christrude 13 अप्रैल 2021

@christrude जबकि मैं null , तो इस पुस्तकालय को रीस्टफुल 404 सम्मेलन को ग्राफक्यूएल null वापसी मूल्य सम्मेलन में परिवर्तित करना चाहिए।

marnusw 13 अप्रैल 2021

@christrude जबकि मैं null , तो इस पुस्तकालय को रीस्टफुल 404 सम्मेलन को ग्राफक्यूएल null वापसी मूल्य सम्मेलन में परिवर्तित करना चाहिए।

फिर ग्राफक्यूएल आपको यूआई में 404 प्रतिक्रिया को संभालने का प्रस्ताव कैसे देता है? कुछ मत करो? तो अनुरोध बस चुपचाप मर जाते हैं? यह अविश्वसनीय रूप से खराब/खराब यूएक्स/एपीआई प्रबंधन है।

christrude 13 अप्रैल 2021

यदि आप किसी आइटम को आईडी द्वारा क्वेरी करते हैं और वापस शून्य हो जाते हैं, तो यूआई में तदनुसार इसे संभाल लें। यदि इसमें डेटा है, तो इसे प्रदर्शित करें।

marnusw 13 अप्रैल 2021

मैं यहां @marnusw से सहमत हूं, हम वास्तव में नियंत्रित नहीं कर सकते हैं कि उपयोगकर्ता का बाकी एपीआई क्या करता है, इसलिए पुस्तकालय को इसकी अनुमति देने में अत्यधिक प्रतिबंधित नहीं होना चाहिए। 404 => नल मैपिंग एक अपेक्षाकृत सहज ज्ञान युक्त अर्थ है जो आपको लापता डेटा पर उड़ाए बिना आरईएसटी के एकल संसाधन प्रति अनुरोध मॉडल से ग्राफक्यूएल के बहु-संसाधन प्रति अनुरोध (क्लाइंट परिप्रेक्ष्य से) में जाने देता है।

शायद इस व्यवहार से ऑप्ट-आउट करने का एक बेहतर तरीका हो सकता है, जैसे प्रतिक्रिया ट्रांसफॉर्मर का पूर्व-पैक संस्करण या अनुरोध प्रबंधन तर्क का एक अमूर्त जो उपयोगकर्ता को http स्थिति प्रबंधन को पूरी तरह से नियंत्रित करने देता है। लेकिन मुझे नहीं लगता कि पिछले व्यवहार को वापस करना सही तरीका है।

paulpdaniels 15 अप्रैल 2021

दृष्टिकोण की समस्या का एक हिस्सा यह है कि परिणाम सदस्यता में नीचे आपको कोई जानकारी नहीं है कि क्या यह एक शून्य परिणाम है क्योंकि यह 404 है, या यदि यह कोई सामग्री नहीं है तो 201 या कुछ समान है। मैंने इसे बाद में तय किया है, स्थिति की जांच कर रहा है और इसे एक त्रुटि फेंकने के लिए मजबूर कर रहा है जो इस मुद्दे को मानता है क्योंकि मैं वास्तविक सर्वर त्रुटि संदेश तक नहीं पहुंच सकता, खासकर मेरे मामले में क्योंकि हम एक ऑथ टोकन इंजेक्ट करने के लिए कस्टम फ़ेच का उपयोग करते हैं , और अपोलो इसे हटाने से पहले प्रतिक्रिया प्राप्त करने के लिए इसका उपयोग नहीं कर सकता। मेरा समाधान एक मामले से अलग, ऐप में अन्य 404 प्रतिक्रियाओं के लिए एक खराब कंबल है।

christrude 15 अप्रैल 2021

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

Apollo-link-rest: 404 त्रुटियों से निपटने के लिए अनुशंसित रणनीति?

सभी 12 टिप्पणियाँ

142 404 त्रुटि प्रबंधन जोड़ता है। मुझे लगता है कि इस चर्चा से यह सबसे महत्वपूर्ण बात है।

संबंधित मुद्दों