Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow HTTP auth pass-though #711

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Conversation

akcoder
Copy link

@akcoder akcoder commented Sep 6, 2013

Allow HTTP auth pass-through. Useful where you have a paranoid systems admin who only trusts HTTP auth to protect the site. Removes the logout link if the user authenticated with HTTP auth.

Also fixes what I presume was a minor oversight where "Howdy, username" was displayed instead of the users full name.

systems admin who only trusts HTTP auth to protect the site.
…sers from HTTP auth. Allow auto-created users to edit their first name, last name email address and a few other pieces of information.
@protich
Copy link
Member

protich commented Sep 11, 2013

Salutation fixed in pull request #713. Opted to show just the first name, full name on salutation might break the page layout for users with long names.

We will revisit authentication is 1.8 or 1.9 - the plan is to provide pluggable authentication plugin system with authentication backend such as HTTP auth, LDAP etc.

@akcoder
Copy link
Author

akcoder commented Sep 11, 2013

Thank you for the update!

On Wed, Sep 11, 2013 at 9:17 AM, Peter Rotich [email protected]:

Salutation fixed in pull request #713#713.
Opted to show just the first name, full name on salutation might break the
page layout for users with long names.

We will revisit authentication is 1.8 or 1.9 - the plan is to provide
pluggable authentication plugin system with authentication backend such as
HTTP auth, LDAP etc.


Reply to this email directly or view it on GitHubhttps://github.com//pull/711#issuecomment-24258616
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants