Closed Bug 920854 Opened 11 years ago Closed 11 years ago

Don't pass empty url into mapper.routematch

Categories

(Cloud Services :: Server: Core, defect)

Product:
Cloud Services
Component:
Server: Core
Platform:
x86_64
Windows 7
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: rfkelly, Unassigned)

Details

(Whiteboard: [qa+])

Attachments

(1 file)

no-empty-url.patch
1.12 KB, patch
telliott
: review+
Details | Diff | Splinter Review 
This was reported by a user in #sync; when request.path_info is empty our url-matching logic throws an exception.
Attachment #810280 - Flags: review?(telliott)
Attachment #810280 - Flags: review?(telliott) → review+
http://hg.mozilla.org/services/server-core/rev/4daa9cf904bb
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Is there some QA love needed here or are we good to go?
Whiteboard: [qa+]
I think we're OK - obscure error case, trivial fix, proper test coverage
Awesome.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: