ymcatwincities / openy_gated_content

Virtual YMCA
https://openy.org
GNU General Public License v3.0
5 stars 28 forks source link

Users with "Virtual YMCA" role see "Access Denied" on password reset attempt #65

Closed froboy closed 3 years ago

froboy commented 3 years ago

To reproduce:

When a user with a member role attempts to reset their password via /user/password they are met with 403 Access Denied. That could be confusing and does not give a path forward if the user unintentionally gets there.

Proposed Resolution: Either create a custom 403 for those Roles or add some boilerplate help text to the 403 page.

hamrant commented 3 years ago

I think this was fixed in https://github.com/fivejars/openy_gated_content/pull/218

@froboy please review

sarah-halby commented 3 years ago

@froboy per @hamrant comment above, can you review the issue and determine if this has been fixed? If so, please close this item.

hamrant commented 3 years ago

This was postponed from release by @anpolimus. PR with fix ready for testing and review

froboy commented 3 years ago

I'll wait until that PR is merged. I tested https://sandbox-carnation-std-virtual-y.openy.org/user/login and I'm seeing a message, but it's only after the user goes back and reloads the form. See the attached video.

https://user-images.githubusercontent.com/238201/105095609-84be5a00-5a6b-11eb-92e9-da84ef1c5c61.mp4

Also, the error message could use some review:

You're not allowing to reset password for user with Virtual Y role.

could be

If you are a Virtual YMCA member, please return to the home page to log in. Contact your branch for login help.

sarah-halby commented 3 years ago

@hamrant just confirming that this item is complete and ready to be closed?

hamrant commented 3 years ago

@sarah-halby yep, the fix was merged 4 days ago