Opened 10 years ago
Closed 10 years ago
#5881 defect closed fixed (fixed)
Fix conch --auth
Reported by: | Ying Li | Owned by: | Ying Li |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | conch | Keywords: | easy |
Cc: | z3p | Branch: |
branches/fix-conch-auth-5881
branch-diff, diff-cov, branch-cov, buildbot |
Author: | cyli |
Description
I broke the --auth option in #4753. Normally it's possible to add multiple credentials checker using the strcred Options mixin. The override of addChecker in conch means that each new --auth option overrides the previous one, so only one checker can be added.
Fix this.
Change History (7)
comment:1 Changed 10 years ago by
Cc: | z3p added |
---|
comment:2 Changed 10 years ago by
Type: | enhancement → defect |
---|
comment:3 Changed 10 years ago by
Author: | → cyli |
---|---|
Branch: | → branches/fix-conch-auth-5881 |
comment:4 Changed 10 years ago by
comment:6 Changed 10 years ago by
Keywords: | review removed |
---|---|
Owner: | set to Ying Li |
Tests pass, fix looks pretty straightforward; merge away.
comment:7 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
(In [35290]) Branching to 'fix-conch-auth-5881'