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

Facebook Error "Circular Redirect Path" #46

Closed
tordans opened this issue Jan 9, 2015 · 4 comments
Closed

Facebook Error "Circular Redirect Path" #46

tordans opened this issue Jan 9, 2015 · 4 comments

Comments

@tordans
Copy link
Contributor

tordans commented Jan 9, 2015

The Facebook Debug Tool (https://developers.facebook.com/tools/debug/og/object/) gives me the following error for our wordpress pages.

I see this error for the first time so I guess facebook changed their behavior!

We have some logik somewhere in wordpress (probably core, since I cannot find any plugins or custom changes about this) that will always redirect you to the version of an url that ends in "/" -> https://www.betterplace.org/c/neues is redirected to https://www.betterplace.org/c/neues/

The bug:
The the og-URL says <meta property="og:url" content="https://www.betterplace.org/c/ueber-uns"/>(without the trailing slash)

How can I fix this?

bildschirmfoto 2015-01-09 um 17 47 38

@chuckreynolds
Copy link
Owner

well they're taking the canonical as a redirect in that regard but hokay. so... we messed with the URL a lot in the develop branch which MAY provide a better solution to the issue? You could try that and see how it does the URL.
Here's the commit that's different from master and hotfix: 269d7c0

chuckreynolds added a commit that referenced this issue Feb 11, 2015
@chuckreynolds
Copy link
Owner

but it does add one here: https://www.betterplace.org/c/neues/digitale-nothilfe-mal-anders-u25/

Reason I bumped this in the recent commit is a few things. I looked at how other plugins and how wordpress are enforcing trailing slashes on urls... and they do it on non-singular pages... so everything that's not a single post, or a single page.

Single posts and single pages typically have a link tag rel="canonical" in the header that wordpress puts in. I don't see that on yours but on single pages the trailing slash works as it should.

I'm wondering if you have another plugin or anything in the theme that's causing an issue with the category pages not having a trailing slash?

....i just noticed this change broke category urls. ugh. I have to look into it more later. I need to sleep.

chuckreynolds added a commit that referenced this issue Jun 19, 2015
touches again on #46 fixed in dev but on hotfix branch for patch worthiness in the wild.
by default this removed trailing spaces in code too.
@chuckreynolds
Copy link
Owner

@gretchdarone
Copy link

hey guys im have this issue. im not good with this can you mabe walk me throw so i can fix mine

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

3 participants