This is an automatically generated e-mail. To reply, visit: http://reviewboard-tflink.rhcloud.com/r/38/ |
On July 17th, 2013, 4:22 p.m. UTC, Tim Flink wrote:
blockerbugs/__init__.py (Diff revision 1) 25 auth = FasLogin(fas)wouldn't it make more sense to have DevLogin here since TEST is used mostly for local integration tests?
Instead of DEV or both, DEV and TEST would have DevLogin? In addition to this, though, we need an environment to test fas-bugzilla account association (we can swith to PROD to test when needed that probably).
On July 17th, 2013, 4:22 p.m. UTC, Tim Flink wrote:
blockerbugs/util/login.py (Diff revision 1) 58 self.user = bunch.Bunch(username='developer', approved_memberships=[])I think it would be better to have the dev user have admin privileges instead of no groups
Good point. Fixed.
On July 17th, 2013, 4:22 p.m. UTC, Tim Flink wrote:
Have you done much testing with FAS when this is enabled? I got a tb on logout when I was poking at actual login (using stg fas). If this code needs fas-openid to work, then maybe we should hold off on merging it to develop until fas-openid is working
Could you point me to the tb? I couldn't reproduce it.
- Martin
On July 18th, 2013, 9:35 a.m. UTC, Martin Krizek wrote:
Review request for blockerbugs.
By Martin Krizek.
Updated July 18, 2013, 9:35 a.m.
Repository:
blockerbugs
Description
Diffs
|