matomo-org / matomo-org/plugin-LoginLdap
Users that are not from LDAP in case of unsuccessful login are checked in LDAP
Open
Nobody has claimed this yet.
- Dominant language
- PHP
- Stars
- 37
- Forks
- 30
- Avg merge
- 22h 53m
- Merged PRs (30d)
- 19
Description
It would be better if users that are not marked as from LDAP (without {LDAP} prefix in password) in case of unsuccessful login are not checked in LDAP and message about bad credentials is displayed instead.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing the unsuccessful-login path described in the issue and identify where users without the {LDAP} password prefix are sent to LDAP. Confirm the existing behavior and authentication tests, if present. Done means such users receive the bad-credentials message without an LDAP lookup, while LDAP-marked users retain their current behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- authentication
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100