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

Login page #7

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Login page #7

wants to merge 12 commits into from

Conversation

cledvina
Copy link
Collaborator

I changed directions on login...

Since we are logging in from three different places, I create a single login page (login.html) and links in the navbars of all three products (homepage, bibframe editor, profile editor).

This page does the usual auth stuff via loopback. For LDAP auth, the script will also create a limited current_user cookie containing the username (since the loopback-passport-component doesn't do this). At least the username is accessible from all pages then.

@cledvina cledvina requested a review from kirkhess January 18, 2019 23:10
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.

1 participant