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)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rtilder, Unassigned)

Details

(Whiteboard: [qa-])

Attachments

(2 files, 1 obsolete file)

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)
Attachment #559219 - Flags: review?(rmiller) → review+
Pushed in http://hg.mozilla.org/services/server-key-exchange/rev/d832b671bcba

Realized that doesn't fix it though.
Attached patch Fix a stupid typo and a thinko (obsolete) — Splinter Review
Attachment #559315 - Flags: review?(rmiller)
A lesson learned: use only one editor per project.
Attachment #559315 - Attachment is obsolete: true
Attachment #559315 - Flags: review?(rmiller)
Attachment #559316 - Flags: review?(rmiller)
Attachment #559316 - Flags: review?(rmiller)
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+
(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].
Fixed in http://hg.mozilla.org/services/server-key-exchange/rev/846cbacd1ac2
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: