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

Laravel-Auditing with Sentinel #62

Closed
AstroBoy269 opened this issue Jun 24, 2016 · 5 comments
Closed

Laravel-Auditing with Sentinel #62

AstroBoy269 opened this issue Jun 24, 2016 · 5 comments

Comments

@AstroBoy269
Copy link

AstroBoy269 commented Jun 24, 2016

We are trying to use laravel-auditng and so far everything seems to be working great except we're not getting any user information populated. We use the Caralyst/Sentinel package for security so our User object extends \Cartalyst\Sentinel\Users\EloquentUser. Any suggestions on what to check?

Error from the laravel.log file:
local.ERROR: exception 'Symfony\Component\Debug\Exception\FatalErrorException' with message 'Access level to OwenIt\Auditing\AuditingTrait::getUserId() must be public (as in class Cartalyst\Sentinel\Users\EloquentUser)' in /chsh/app/Objects/User.php:473

Thanks in advance.

@tyteen4a03
Copy link

tyteen4a03 commented Jul 6, 2016

There seems to be no provision to support Sentinel (or any other 3rd party packages) because it makes calls to the Auth package directly. I've opened #65.

@anteriovieira
Copy link
Member

Hello guys, first sorry for the absence.

I will try to meet everyone as fast as possible. We are also open to pull request when they feel necessary.

For now maybe this can help #40

@tyteen4a03
Copy link

You also need to change the function's name to something else, since it clashes with Sentinel's own function. For me, I've changed it to getLoggedInUserId.

@anteriovieira
Copy link
Member

Cool, you can open a pull request please.

@anteriovieira
Copy link
Member

Sorry for the delay #40 .

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

No branches or pull requests

3 participants