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

Urls in catched mails are demaged #17

Closed
mediaessenz opened this issue May 17, 2024 · 1 comment · Fixed by #22
Closed

Urls in catched mails are demaged #17

mediaessenz opened this issue May 17, 2024 · 1 comment · Fixed by #22
Assignees

Comments

@mediaessenz
Copy link

mediaessenz commented May 17, 2024

I would like to use this extension to catch mails on a staging system, which are generated by a registration process.
The generated mail contain a link to confirm a registration (double opt in).
Unfortunately the links in the mails are damaged and do not work as expected.

A working link should look like this:
https://domain/sign-in/registration/confirmation?tx_getaccess_confirmation%5Baction%5D=user&tx_getaccess_confirmation%5Bcontroller%5D=Confirmation&tx_getaccess_confirmation%5Bhash%5D=2720ba93a3007066&cHash=3179e61306999063853b2247a1bd4d

But in the catched mail inside of the backend module of this extension the same link look like this:
https://domain/sign-in/registration/confirmation?tx_getaccess_confirmation%5Baction%5D=user&tx_getaccess_confirmation%5Bcontroller%5D=Confirmation&tx_getaccess_confirmation%5Bhash%5D%2720ba93a3007066&cHash^3179e61306999063853b2247a1bd4d

As you can see, there are some small, but important differences:
The equal (=) sign after the hash value became a percentage (%) sign. And the equal (=) sign of the cHash become a ^ sign

Because of this, the registration process fails.

@maikschneider maikschneider self-assigned this May 27, 2024
This was referenced May 27, 2024
@maikschneider
Copy link
Member

Hi @mediaessenz, thanks for reporting! I think I found the issue. Could you please test v1.3.3?

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

Successfully merging a pull request may close this issue.

2 participants