jotaelesalinas / laravel-adminless-ldap-auth

Authenticate users in Laravel against an adminless LDAP server
MIT License
210 stars 33 forks source link

Use Authorizable trait in LdapUser to facilitate use with authorizati… #54

Closed LichP closed 3 years ago

LichP commented 3 years ago

…on policies [refs #53]

Motivation and context

https://github.com/jotaelesalinas/laravel-adminless-ldap-auth/issues/53

How has this been tested?

Tested with a simple policy for an eloquent model, example function:

    /**
     * Determine whether the user can view the model.
     *
     * @param  \JotaEleSalinas\AdminlessLdap\LdapUser  $
     * @param  \App\Models\Candidate  $candidate
     * @return mixed
     */
    public function view(LdapUser $user, Candidate $candidate)
    {
        return $user->username == $candidate->uname;
    }

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

Checklist:

Go over all the following points, and put an x in all the boxes that apply.

Please, please, please, don't send your pull request until all of the boxes are ticked. Once your pull request is created, it will trigger a build on our continuous integration server to make sure your tests and code style pass.

If you're unsure about any of these, don't hesitate to ask. We're here to help!