Problem

If we set the About page as canonical page and go to this page as outsider we will get an error that says that we were redirected too many times.

Proposed resolution

If the About page was set as canonical and user does not have access to this group, show "Access denied" page instead of redirect from canonical page to About page.

CommentFileSizeAuthor
#3 cyclic-redirection-2894571-3.patch3.41 KBalex.ksis
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alex.ksis created an issue. See original summary.

alex.ksis’s picture

alex.ksis CreditAttribution: alex.ksis as a volunteer and at Lemberg Solutions, Open Social for Open Social commented
Issue summary: View changes
alex.ksis’s picture

alex.ksis CreditAttribution: alex.ksis as a volunteer and at Lemberg Solutions, Open Social for Open Social commented
Status: Active » Needs review
FileSize
cyclic-redirection-2894571-3.patch3.41 KB

Here is the patch and the PR for it: https://github.com/goalgorilla/open_social/pull/473

frankgraave’s picture

frankgraave CreditAttribution: frankgraave for Open Social commented
Status: Needs review » Closed (won't fix)

Closing this issue. As stated in my comment on the pull request I think it a very uncommon use case that the group canonical is being changed and therefore the fix should be done within a project where the group canonical is being changed to a custom value.

Further information and arguments on this can be found in the pull request comments.

Cheers, Frank