Project

General

Profile

Actions

Bug #6700

closed

[PATCH] Fix some 4xx status classes to inherit correctly from Net::HTTPClientError

Added by madeofcode (Mark Dodwell) over 12 years ago. Updated over 12 years ago.

Status:
Closed
Target version:
ruby -v:
ruby 2.0.0dev (2012-07-05 trunk 36311) [x86_64-darwin11.4.2]
Backport:
[ruby-core:46189]

Description

These recently added 4xx status classes were inheriting from Net::HTTPSuccess, which seems incorrect.

I believe they should inherit from Net::HTTPClientError, as with all the other 4xx cases.

Note that this change subtly alters/fixes the behavior of these classes since they will now inherit the EXCEPTION_TYPE = Net::HTTPServerException from Net::HTTPClientError which they would not have had previously. I expect that's disable, but just wanted to flag that.


Files

Updated by madeofcode (Mark Dodwell) over 12 years ago

s/disable/desirable/

Updated by knu (Akinori MUSHA) over 12 years ago

  • Status changed from Open to Assigned
  • Assignee set to knu (Akinori MUSHA)
Actions #3

Updated by knu (Akinori MUSHA) over 12 years ago

  • Status changed from Assigned to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r36316.
Mark, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


  • lib/net/http/responses.rb: Fix 4xx classes to inherit correctly
    from Net::HTTPClientError. [Bug #6700]

Updated by knu (Akinori MUSHA) over 12 years ago

Thanks for pointing it out. I merged your patch and added a test.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0