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

Error being emailed out. #3

Closed
MattyBalaam opened this issue Aug 12, 2016 · 5 comments
Closed

Error being emailed out. #3

MattyBalaam opened this issue Aug 12, 2016 · 5 comments

Comments

@MattyBalaam
Copy link
Contributor

[Error] User needs to be logged in access multi-user editing data
/var/lib/silverstripe/www.neweraed.co.uk.btn.dbplc.com/multiuser-editing/code/MultiUserEditingController.php:20

@dhensby
Copy link
Contributor

dhensby commented Aug 12, 2016

Someone's trying to access the controller whilst not being logged in

@dhensby dhensby closed this as completed Aug 12, 2016
@MattyBalaam
Copy link
Contributor Author

I guess I don't understand why. These errors wouldn't normally be sent out if someone was logged out would they?

@dhensby
Copy link
Contributor

dhensby commented Aug 12, 2016

the controller is probably having the init function run before the canView or other security checks are happening.

@MattyBalaam
Copy link
Contributor Author

OK, so are you saying this would not happen on a base install, and another module or some custom code is causing this issue?

@dhensby
Copy link
Contributor

dhensby commented Sep 13, 2016

From what I see the only way to access this controller is by sending a request to /admin/editing/$ID. That URL is called by a JS script that is loaded in the admin area.

The only way I can replicate this error is to login to the admin, then logout in another tab and then the ping hits the server and reveals that error.

I admit it would be nicer just to throw a permissions error and to trigger the login modal in the CMS, but I can't work on that at the moment.

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

2 participants