Bug #6700

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

Added by Mark Dodwell almost 3 years ago. Updated almost 3 years ago.

[ruby-core:46189]
Status:Closed
Priority:Normal
Assignee:Akinori MUSHA
ruby -v:ruby 2.0.0dev (2012-07-05 trunk 36311) [x86_64-darwin11.4.2] Backport:

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.

0001-Fix-4xx-classes-to-inherit-correctly-from-Net-HTTPCl.patch Magnifier (1.56 KB) Mark Dodwell, 07/05/2012 06:49 AM

Associated revisions

Revision 36316
Added by Akinori MUSHA almost 3 years ago

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

Revision 36316
Added by Akinori MUSHA almost 3 years ago

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

History

#1 Updated by Mark Dodwell almost 3 years ago

s/disable/desirable/

#2 Updated by Akinori MUSHA almost 3 years ago

  • Status changed from Open to Assigned
  • Assignee set to Akinori MUSHA

#3 Updated by Akinori MUSHA almost 3 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]

#4 Updated by Akinori MUSHA almost 3 years ago

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

Also available in: Atom PDF