Closed
Bug 371861
Opened 17 years ago
Closed 17 years ago
Need longer location entry text area
Categories
(Webtools :: Bouncer, defect)
Webtools
Bouncer
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: preed, Assigned: morgamic)
References
Details
Attachments
(1 file)
457 bytes,
patch
|
morgamic
:
review+
|
Details | Diff | Splinter Review |
On the file location text area on http://download.mozilla.org/admin/locations.php, there's a maxlength of 100 characters. With the partner builds, we're running into that limit, and now can no longer serve partial updates and complete updates for Linux (because their path specification is over 100 characters). I'm hoping this is just some weird UI thing, and we can fix it easily, and not a database constraint. You're our only hope, morgamic...
Reporter | ||
Comment 1•17 years ago
|
||
(BTW, this is blocker because it's blocking the release of 1.5.0.10 partner updates.)
Comment 2•17 years ago
|
||
I have no idea how where this limit is implemented, but we should make sure we investigate the web interface as well as any back-end database and/or configuration that might be affected by a change to that limit. I don't think I can handle any more bustage with this release...
Updated•17 years ago
|
OS: Linux → All
Hardware: PC → All
Comment 3•17 years ago
|
||
Change the max to 255 to match the maximum length allowed in the db.
Updated•17 years ago
|
Attachment #256558 -
Flags: review?(morgamic)
Assignee | ||
Updated•17 years ago
|
Attachment #256558 -
Flags: review?(morgamic) → review+
Assignee | ||
Comment 4•17 years ago
|
||
Jay, the form input was set to 100 max, but the db dump from production has a max length of 255. The patch lets them match up. There's no unknown regression from this simple change. Thanks Reed for posting the fix! You da man! I've checked in the patch and emailed oncall. Should be updated shortly. IT: need to update the bouncer1 tag on the OSL cluster. It should already be pulling from the 1.0-prod-02082007 tag.
Assignee | ||
Comment 5•17 years ago
|
||
Was updated in production and I verified that we can add a 100+ character location_path.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•