Closed
Bug 685605
Opened 13 years ago
Closed 13 years ago
Replace references to obsolete functions convert_config and filter_params
Categories
(Cloud Services :: Server: Key Exchange, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: rtilder, Unassigned)
Details
(Whiteboard: [qa-])
Attachments
(2 files, 1 obsolete file)
751 bytes,
patch
|
rmiller
:
review+
|
Details | Diff | Splinter Review |
890 bytes,
patch
|
rmiller
:
review+
rmiller
:
review+
|
Details | Diff | Splinter Review |
Jenkins has been complaining about the key-exchange builds failing. This is at least one part of it. keyexchange/wsgiapp.py needs to use the newer Config object instead of convert_config and filter_params.
Attachment #559219 -
Flags: review?(rmiller)
Updated•13 years ago
|
Attachment #559219 -
Flags: review?(rmiller) → review+
Reporter | ||
Comment 1•13 years ago
|
||
Pushed in http://hg.mozilla.org/services/server-key-exchange/rev/d832b671bcba Realized that doesn't fix it though.
Reporter | ||
Comment 2•13 years ago
|
||
Attachment #559315 -
Flags: review?(rmiller)
Reporter | ||
Comment 3•13 years ago
|
||
A lesson learned: use only one editor per project.
Attachment #559315 -
Attachment is obsolete: true
Attachment #559315 -
Flags: review?(rmiller)
Reporter | ||
Updated•13 years ago
|
Attachment #559316 -
Flags: review?(rmiller)
Reporter | ||
Updated•13 years ago
|
Attachment #559316 -
Flags: review?(rmiller)
Comment 4•13 years ago
|
||
Comment on attachment 559316 [details] [diff] [review] Fix for stupid type and two thinkos Review of attachment 559316 [details] [diff] [review]: ----------------------------------------------------------------- this makes sense when taken cumulatively w/ the last patch (https://bugzilla.mozilla.org/page.cgi?id=splinter.html&bug=685605&attachment=559219)
Attachment #559316 -
Flags: review?(rmiller)
Attachment #559316 -
Flags: review+
Reporter | ||
Comment 5•13 years ago
|
||
(In reply to Ryan Tilder [:rtilder] from comment #3) > Created attachment 559316 [details] [diff] [review] > Fix for stupid type and two thinkos > Should make a note that these changes are cumulative with the attachment 559219 [details] [diff] [review].
Reporter | ||
Comment 6•13 years ago
|
||
Fixed in http://hg.mozilla.org/services/server-key-exchange/rev/846cbacd1ac2
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•13 years ago
|
Whiteboard: [qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•