Closed
Bug 775694
Opened 13 years ago
Closed 13 years ago
Peptest should allow multiple --proxy commands
Categories
(Testing Graveyard :: Peptest, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ahal, Assigned: ahal)
References
Details
Attachments
(1 file)
|
5.03 KB,
patch
|
mcote
:
review+
|
Details | Diff | Splinter Review |
At the moment --proxy is a string, but it should be converted to an array using optparse's 'append' attribute. This would allow us to specify multiple server-locations.txt rather than requiring all proxied sites to be defined in one.
|
Assignee | |
Updated•13 years ago
|
Assignee: nobody → ahalberstadt
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 1•13 years ago
|
||
I also switched to string substitutions in pepprocess.py since I didn't think it was worth filing a separate bug about.
Attachment #644417 -
Flags: review?(mcote)
|
||
Comment 2•13 years ago
|
||
Comment on attachment 644417 [details] [diff] [review]
Patch 1.0 - Allow multiple --proxies
Cool. Might want to mention in the help string for --proxy that it can be specified multiple times.
Attachment #644417 -
Flags: review?(mcote) → review+
|
|
Assignee | |
Comment 3•13 years ago
|
||
master: https://github.com/mozilla/peptest/commit/6a056d8c4ab60dac9f7b03c5d35285ef3c0c143f
This will need to be mirrored to m-c, but I'm going to wait until the bustage in bug 774817 is sorted out as it will probably also require changes to m-c.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
|
||
Updated•7 years ago
|
Product: Testing → Testing Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•