backward compatibility for old authtype settings #199

Merged
merged 6 commits into from Apr 7, 2013

Conversation

Projects
None yet
4 participants
Owner

splitbrain commented Mar 17, 2013

I think the message should displayed to superusers only. Otherwise I'm fine with merging

Collaborator

michitux commented Mar 17, 2013

+1 (for superuser only + merging)

Owner

splitbrain commented Mar 17, 2013

Hmm. Easier said than done. At that point in the code the identity of the current user is not established, yet.

Collaborator

bug commented Mar 20, 2013

If not only the superuser sees the message, then the setting will be updated quickly :-) I'd go for not adding too much code to get the message displayed to the superuser only.

Collaborator

Chris--S commented Apr 1, 2013

See PR#210 for superuser only message.

#210

@Chris--S Chris--S added a commit that referenced this pull request Apr 7, 2013

@Chris--S Chris--S Merge pull request #199 from splitbrain/authcomp
backward compatibility for old authtype settings
33f1e87

@Chris--S Chris--S merged commit 33f1e87 into master Apr 7, 2013

Chris--S deleted the authcomp branch Apr 7, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment