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

URL tracking dont work if the URL has an ampersand in dynamic content #9278

Closed
wise-gorilla opened this issue Oct 9, 2020 · 6 comments
Closed
Labels
bug Issues or PR's relating to bugs dynamic-content email Anything related to email stale Issues which have not received an update within 90 days T1 Low difficulty to fix (issue) or test (PR)

Comments

@wise-gorilla
Copy link
Contributor

wise-gorilla commented Oct 9, 2020

Bug Description

That issue got solved on this PR #2060 but only for the urls added directly inside the email however it didn't solve the issue with the urls added inside a dynamic content

Q A
Mautic version 3.1.1
PHP version 7.3
Browser Chrome

Steps to reproduce

  1. Add a url with an ampersand like https://example.com/?foo=bar&bar=foo inside a dynamic content
  2. Use this dynamic content inside an email by ex. Click here

Log errors

@wise-gorilla wise-gorilla added the needs-triage For new issues/PRs that need to be triaged label Oct 9, 2020
@npracht
Copy link
Member

npracht commented Oct 16, 2020

Hi @wise-gorilla would you please test #8345 and tell if it fixes the issue ?

@npracht npracht added bug Issues or PR's relating to bugs email Anything related to email pending-feedback PR's and issues that are awaiting feedback from the author and removed needs-triage For new issues/PRs that need to be triaged labels Oct 16, 2020
@wise-gorilla
Copy link
Contributor Author

@npracht I tested that PR and unfortuinatly it don't solve that issue

@npracht npracht added dynamic-content and removed pending-feedback PR's and issues that are awaiting feedback from the author labels Oct 17, 2020
@npracht
Copy link
Member

npracht commented Oct 17, 2020

@kuzmany maybe you can tackle that in your PR ?

@RCheesley RCheesley added the T1 Low difficulty to fix (issue) or test (PR) label Dec 11, 2020
@stale
Copy link

stale bot commented Apr 25, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Issues which have not received an update within 90 days label Apr 25, 2021
@stale
Copy link

stale bot commented May 9, 2021

This issue has been automatically closed because it has not had recent activity. If the reported issue persists, please create a new issue and link back to this one for reference. Thank you for your contributions.

@stale stale bot closed this as completed May 9, 2021
@arturromaorocha
Copy link

arturromaorocha commented May 3, 2022

I have the same problem, mautic only identifies the first get parameter and ignores the rest.

://site?foo=bar&bar=foo&other=...
only foo is sent to the destination

v4.2.2

PHP Version 7.4.3

Database info
Version 8.0.28-0ubuntu0.20.04.3
Doctrine driver pdo_mysql

image

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issues or PR's relating to bugs dynamic-content email Anything related to email stale Issues which have not received an update within 90 days T1 Low difficulty to fix (issue) or test (PR)
Projects
None yet
Development

No branches or pull requests

4 participants