Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
in my actual use case, the user trying to authenticate through oauth and returning to oauth/authorized (still anonymous at that stage) has not (and should not have) the "access content" permission.
I suggest that we have a specific permission for that menu entry.
Comment | File | Size | Author |
---|---|---|---|
#3 | oauth-authorized_callback_permission-2102793-3.patch | 909 bytes | Nitesh Sethia |
#1 | oauth-authorized_callback_permission-2102793-1.patch | 845 bytes | garphy |
Comments
Comment #1
garphy CreditAttribution: garphy commentedPatch.
Comment #2
MustangGB CreditAttribution: MustangGB commentedYes please!
My only comment is that it should appear first in the
hook_permission
list, rather than last, for consistency with other permissions normally the "safer" ones are at the top, and the "restricted" ones are at the bottom.Comment #3
Nitesh Sethia CreditAttribution: Nitesh Sethia as a volunteer and commentedUpdated the patch as per the last comment.