WebException วิธีรับการตอบสนองทั้งหมดกับร่างกาย?


110

ใน WebException ฉันไม่เห็นเนื้อความของ GetResponse นี่คือรหัสของฉันใน C #:

try {                
  return GetResponse(url + "." + ext.ToString(), method, headers, bodyParams);
} catch (WebException ex) {
    switch (ex.Status) {
      case WebExceptionStatus.ConnectFailure:
         throw new ConnectionException();                        
     case WebExceptionStatus.Timeout:
         throw new RequestTimeRanOutException();                     
     case WebExceptionStatus.NameResolutionFailure:
         throw new ConnectionException();                        
     case WebExceptionStatus.ProtocolError:
          if (ex.Message == "The remote server returned an error: (401) unauthorized.") {
              throw new CredentialsOrPortalException();
          }
          throw new ProtocolErrorExecption();                    
     default:
          throw;
    }

ฉันเห็นส่วนหัว แต่ไม่เห็นร่างกาย นี่คือเอาต์พุตจาก Wireshark สำหรับคำขอ:

POST /api/1.0/authentication.json HTTP/1.1    
Content-Type: application/x-www-form-urlencoded    
Accept: application/json    
Host: nbm21tm1.teamlab.com    
Content-Length: 49    
Connection: Keep-Alive    

userName=XXX&password=YYYHTTP/1.1 500 Server error    
Cache-Control: private, max-age=0    
Content-Length: 106    
Content-Type: application/json; charset=UTF-8    
Server: Microsoft-IIS/7.5    
X-AspNet-Version: 2.0.50727    
X-Powered-By: ASP.NET    
X-Powered-By: ARR/2.5

Date: Mon, 06 Aug 2012 12:49:41 GMT    
Connection: close    

{"count":0,"startIndex":0,"status":1,"statusCode":500,"error":{"message":"Invalid username or password."}}

เป็นไปได้ไหมที่จะเห็นข้อความใน WebException ขอบคุณ.


คุณได้ลอง (HttpWebResponse) we.Response หรือยัง 'เรา' อยู่ที่ไหน WebException ของคุณ?
Justin Harvey

2
ในการรักษาสแต็กแทร็กในข้อยกเว้น rethrown อย่าใช้throw ex;แต่เพียงthrow;(ในกรณีเริ่มต้น) นอกจากนี้ (ถ้าจำเป็น) ฉันจะใส่ WebException ดั้งเดิมใน InnerException ของคุณเอง (ผ่านตัวสร้างที่เหมาะสม)
user1713059

คำตอบ:


202
var resp = new StreamReader(ex.Response.GetResponseStream()).ReadToEnd();

dynamic obj = JsonConvert.DeserializeObject(resp);
var messageFromServer = obj.error.message;

8
สำหรับใครที่ไม่คุ้นเคยกับ JsonConvert คุณต้องรับ Newtonsoft.Json จาก nuget package manager
Kyle

โปรดอัปเดตคำตอบด้วยคำอธิบายของ Kyle เนื่องจาก Newtonsoft.Json เป็น optionall
Jeroen

3
นอกจากนี้โปรดอธิบายว่ารหัสนี้ควรอยู่ในส่วนคำสั่ง Catch ทางเลือกของบล็อกโค้ด Try-Catch ที่คำขอควรไป ฉันรู้ว่าในกรณีนี้เป็นที่ชัดเจนสำหรับผู้อ่านที่ให้ความสนใจและ @iwtu แต่คำตอบที่ครอบคลุมอย่างครบถ้วนสามารถสร้างความแตกต่างอย่างแท้จริงให้กับผู้เริ่มต้นอ่านคำตอบนี้)
Jeroen

2
StreamReader ใช้ IDisposable ดังนั้นจึงไม่ใช่แนวทางปฏิบัติที่ดีที่สุดในการรวมสิ่งนี้ไว้ในคำสั่งโดยใช้? ดูวิธีการ Dispose ของ StreamReader อย่างรวดเร็วแสดงให้เห็นว่ามีการล้างข้อมูลที่สำคัญบางอย่างในนั้น
sammy34

@ sammy34 ไม่ต้องกังวลเนื่องจากในกรณีนี้ไม่มีรหัส / ข้อมูลที่ไม่มีการจัดการผู้เก็บขยะสามารถจัดการได้อย่างง่ายดาย ... (แต่การใช้โดยใช้เป็นนิสัยที่ดีเสมอ)
LB

42
try {
 WebClient client = new WebClient();
 client.Encoding = Encoding.UTF8;
 string content = client.DownloadString("https://sandiegodata.atlassian.net/wiki/pages/doaddcomment.action?pageId=524365");
 Console.WriteLine(content);
 Console.ReadKey();
} catch (WebException ex) {
 var resp = new StreamReader(ex.Response.GetResponseStream()).ReadToEnd();
 Console.WriteLine(resp);
 Console.ReadKey();
}

5

สิ่งนี้จะปรับปรุงเฉพาะคำตอบที่มีอยู่ ฉันได้เขียนวิธีการที่ดูแลรายละเอียดของการโยน / การปลูกใหม่ด้วยข้อความที่ปรับปรุงแล้วซึ่งรวมถึงเนื้อหาการตอบสนอง:

นี่คือรหัสของฉัน (ใน Client.cs):

/// <summary>
///     Tries to rethrow the WebException with the data from the body included, if possible. 
///     Otherwise just rethrows the original message.
/// </summary>
/// <param name="wex">The web exception.</param>
/// <exception cref="WebException"></exception>
/// <remarks>
///     By default, on protocol errors, the body is not included in web exceptions. 
///     This solutions includes potentially relevant information for resolving the
///     issue.
/// </remarks>
private void ThrowWithBody(WebException wex) {
    if (wex.Status == WebExceptionStatus.ProtocolError) {
        string responseBody;
        try {
            //Get the message body for rethrow with body included
            responseBody = new StreamReader(wex.Response.GetResponseStream()).ReadToEnd();

        } catch (Exception) {
            //In case of failure to get the body just rethrow the original web exception.
            throw wex;
        }

        //include the body in the message
        throw new WebException(wex.Message + $" Response body: '{responseBody}'", wex, wex.Status, wex.Response);
    }

    //In case of non-protocol errors no body is available anyway, so just rethrow the original web exception.
    throw wex;
}

คุณใช้มันในประโยค catch เหมือนกับที่ OP แสดง:

//Execute Request, catch the exception to eventually get the body
try {
    //GetResponse....
    }
} catch (WebException wex) {
    if (wex.Status == WebExceptionStatus.ProtocolError) {
        ThrowWithBody(wex);
    }

    //otherwise rethrow anyway
    throw;
}
โดยการใช้ไซต์ของเรา หมายความว่าคุณได้อ่านและทำความเข้าใจนโยบายคุกกี้และนโยบายความเป็นส่วนตัวของเราแล้ว
Licensed under cc by-sa 3.0 with attribution required.