???? #1

New Issue

Closed

opened 2018-11-18 18:29:52 +01:00 by nightpool · 3 comments

nightpool commented

2018-11-18 18:29:52 +01:00

Member

https://cybre.tech/cybrespace/mastodon/commit/df0855a91b596f3bd2d4d4c9bced7a30b201ed1b ![](https://i.imgur.com/UIvGFX5.png)

Alamantus commented

2018-11-18 18:42:41 +01:00

Extremely important whitespace!

Extremely important whitespace!

chr commented

2018-11-20 07:29:15 +01:00

Owner

not a bug

not a bug

chr closed this issue

2018-11-20 07:29:16 +01:00

nightpool commented

2018-11-24 02:30:55 +01:00

Author

Member

:/

:/

chr referenced this issue from a commit

2020-02-16 22:05:47 +01:00

LDAP & PAM added to OAuth password grant strategy (#7999) (#12390) When authenticating via OAuth, the resource owner password grant strategy is allowed by Mastodon, but (without this PR), it does not attempt to authenticate against LDAP or PAM. As a result, LDAP or PAM authenticated users cannot sign in to Mastodon with their email/password credentials via OAuth (for instance, for native/mobile app users). This PR fleshes out the authentication strategy supplied to doorkeeper in its initializer by looking up the user with LDAP and/or PAM when devise is configured to use LDAP/PAM backends. It attempts to follow the same logic as the Auth::SessionsController for handling email/password credentials. Note #1: Since this pull request affects an initializer, it's unclear how to add test automation. Note #2: The PAM authentication path has not been manually tested. It was added for completeness sake, and it is hoped that it can be manually tested before merging.

Sign in to join this conversation.

No Label

No Milestone

No Assignees

3 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cybrespace/mastodon#1

No description provided.