Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

This project is archived and is in readonly mode.

A sane exception should be raised for missing Location header

#1192

If you supply "" or just don't supply a Location header at all, ActiveResource raises a very ugly, unintelligible exception.

Reported by Bob Aman · October 8th, 2008 @ 05:26 PM

State: resolved
Milestone: 2.x
Assigned to: Pratik Pratik
Importance: none

Activity

  1. Pratik
    Pratik
    • Milestone cleared.
    • Assigned user set to Pratik
    • Title changed from [PATCH] A sane exception should be raised for missing Location header to A sane exception should be raised for missing Location header

    January 18th, 2009 @ 06:53 AM

  2. Pratik
    Pratik
    • Milestone set to 2.x

    Hey,

    The patch doesn't apply clean :(

    January 28th, 2009 @ 07:56 PM

  3. Bob Aman
    Bob Aman

    Bah, it used to. I'll update it.

    January 28th, 2009 @ 08:20 PM

  4. Bob Aman
    Bob Aman

    Any chance the patch can be applied before it goes stale this time? :-/

    January 28th, 2009 @ 08:46 PM

  5. Repository
    Repository
    • State changed from new to resolved

    (from [166c63818e901e64786a76029febf7c9cdb40f2d]) Improve exception handling when Location header is invalid. [#1192 A sane exception should be raised for missing Location header state:resolved]

    Signed-off-by: Pratik Naik pratiknaik@gmail.com http://github.com/rails/rails/co...

    January 28th, 2009 @ 08:59 PM