Page MenuHomePhabricator

RfC: AuthManager
Closed, ResolvedPublic

Description

Discuss and decide on AuthManager RfC

Event Timeline

bd808 raised the priority of this task from to Needs Triage.
bd808 updated the task description. (Show Details)
bd808 added a subscriber: bd808.
daniel reassigned this task from Legoktm to bd808.
daniel moved this task from P1: Define to Under discussion on the TechCom-RFC board.
daniel added a subscriber: Legoktm.
daniel added a subscriber: daniel.

@daniel We would like to request an IRC discussion of this RfC at the earliest convenience of the committee

Action items from the IRC meeting:

  • SMalyshev is concerned about lack of model/controller split (TimStarling, 21:32:16)
  • several people would like to see interface definitions (TimStarling, 21:32:34)
  • no serious problems in principle given the current level of detail (TimStarling, 21:33:52)
This comment was removed by Spage.

@Anomie has a work in progress patch at https://gerrit.wikimedia.org/r/#/c/195297/ which defines interfaces and is starting to flesh out additional details of the implementation.

@csteipp has produced a state diagram of the proposed workflow that is now included in the RfC.

bd808 triaged this task as Medium priority.Apr 9 2015, 12:27 AM

Can this be marked resolved, or should we have another round of comments? AuthManager is pretty close to being fully implemented now.

Yeah, let's close this one.

FYI the RfC was never marked approved on the wiki page nor in TechCom-RFC. on the RfC text, see T89459#2289662 .