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

Ignore new Facebook Url Parameter - fbclid #649

Closed
WorksDev opened this issue Oct 21, 2018 · 5 comments
Closed

Ignore new Facebook Url Parameter - fbclid #649

WorksDev opened this issue Oct 21, 2018 · 5 comments

Comments

@WorksDev
Copy link
Contributor

@WorksDev WorksDev commented Oct 21, 2018

No description provided.

@dmitryd
Copy link
Owner

@dmitryd dmitryd commented Oct 23, 2018

Thank you!

@Outdoorsman
Copy link

@Outdoorsman Outdoorsman commented Nov 6, 2018

This is a breaking change on a lot of websites... just a +1 for getting this into a version update so we don't have to go through every website to use dev-development only to revert again.

@dhuf
Copy link

@dhuf dhuf commented Nov 6, 2018

@Outdoorsman I agree, but to fix it I used the composer-patch in case it might help

@uocnma
Copy link

@uocnma uocnma commented Dec 17, 2018

Is it planned to release a version 2.4.1 to TER which includes this fix ? We don't have the possibility to install via composer. realURL 2.4 does not include the fix yet.

Kind regards,
Nelson.

@sypets
Copy link

@sypets sypets commented Dec 17, 2018

On a related note: What I did is remove fbclid entirely in Webserver configuration entirely, for example in Apache:

RewriteCond %{QUERY_STRING} ^(.*&)?fbclid=[^&]*(?:&(.*))?$ [NC]
RewriteRule ^ %{REQUEST_URI}?%1%2 [L,R=301,NE]

Unless you plan to use this in some way, I see no benefit of passing this along to TYPO3 in the first place. Benefits:

  • use redirects, so you get final URL without fbclid and prevent people linking to this
  • works whatever version of realurl or other URL generation you currently use.
  • simple solution that requires no extra configuration in TYPO3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
6 participants
You can’t perform that action at this time.