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

Why back method needs authorization? #133

Open
lcjury opened this issue Jul 10, 2023 · 1 comment
Open

Why back method needs authorization? #133

lcjury opened this issue Jul 10, 2023 · 1 comment

Comments

@lcjury
Copy link

lcjury commented Jul 10, 2023

I want to allow admins to masquerade other users, because of this, I override the masquerade_authorized? method with something like:

def masquerade_authorized?
  admin?
end

The "back_masquerade_path" also uses masquerade_authorized?; since I'm masquerading as a non-admin user, I cannot return to my previous user.

I'm doing something wrong?

@MartinSugasti
Copy link

You could do something like

def masquerade_authorized?
  admin? || params[:action] == 'back'
end

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