diff options
author | alecdwm | 2016-12-13 22:31:35 +0100 |
---|---|---|
committer | alecdwm | 2016-12-13 22:41:07 +0100 |
commit | 02e99277146d8bd912f2f19af1d3e94a6181d90d (patch) | |
tree | 01140b215f4defc46ec28f599d27ebe31dbf2915 /lib/response.js | |
parent | 8095f8cc983bbed532db7eee9f7f5a64e910b731 (diff) |
Initial support for LDAP server authentication
Limitations as of this commit:
- tlsOptions can only be specified in config.json, not as env vars
- authentication failures are not yet gracefully handled by the UI
- instead the error message is shown on a blank page (/auth/ldap)
- no email address is associated with the LDAP user's account
- no picture/profile URL is associated with the LDAP user's account
- we might have to generate our own access + refresh tokens,
because we aren't using oauth. The currently generated
tokens are just a placeholder.
- 'LDAP Sign in' needs to be translated to each locale
Diffstat (limited to 'lib/response.js')
-rwxr-xr-x | lib/response.js | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/lib/response.js b/lib/response.js index aae39851..f0f49181 100755 --- a/lib/response.js +++ b/lib/response.js @@ -66,6 +66,7 @@ function showIndex(req, res, next) { gitlab: config.gitlab, dropbox: config.dropbox, google: config.google, + ldap: config.ldap, email: config.email, signin: req.isAuthenticated(), infoMessage: req.flash('info'), @@ -98,6 +99,7 @@ function responseHackMD(res, note) { gitlab: config.gitlab, dropbox: config.dropbox, google: config.google, + ldap: config.ldap, email: config.email }); } |