Closed
Bug 890669
Opened 12 years ago
Closed 12 years ago
Entering a capital in title/username search breaks search
Categories
(Webmaker Graveyard :: webmaker.org, defect)
Webmaker Graveyard
webmaker.org
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sarah.lynn.89, Assigned: mjschranz)
References
Details
Attachments
(1 file)
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:22.0) Gecko/20100101 Firefox/22.0 (Beta/Release)
Build ID: 20130618035212
Steps to reproduce:
Entered capital letters at the start of a title/username search and clicked search
Actual results:
When page loaded it said {"code" : "ECONNRESET"}
Comment 1•12 years ago
|
||
I can confirm this (I was there :)
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 2•12 years ago
|
||
I'm getting a page with an empty object
`{}`
STR:
Open search page /search
Search by "Title"
Typed "Sneak"
https://webmaker.org/search?type=title&q=Sneak
Comment 3•12 years ago
|
||
User search produces the result in the Description
Title search produces the result in Comment 2
| Assignee | ||
Updated•12 years ago
|
Assignee: nobody → schranz.m
| Assignee | ||
Comment 4•12 years ago
|
||
User search is related to Bug 890557. I'll add more to that bug by making sure it lowercases the values passed in for username since that's what we store in the login DB.
| Assignee | ||
Comment 5•12 years ago
|
||
Actually no. I'm going to do that in the login side of things.
| Assignee | ||
Comment 6•12 years ago
|
||
Login Server patch for first issue.
Attachment #771847 -
Flags: review?(kieran.sedgwick)
Updated•12 years ago
|
Attachment #771847 -
Flags: review?(kieran.sedgwick) → review+
Comment 7•12 years ago
|
||
Commit pushed to master at https://github.com/mozilla/login.webmaker.org
https://github.com/mozilla/login.webmaker.org/commit/9c55ca5a567b4350d37c22e07a2f89c6ae8f9b95
Bug 890669 - Lowercase all usernames when used as a paramater for retrieving a user record
| Assignee | ||
Comment 9•12 years ago
|
||
Both issues appear to be fixed on production, so I'm going to close this up.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Attachment mime type: text/plain → text/x-github-pull-request
You need to log in
before you can comment on or make changes to this bug.
Description
•