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

Security Update #1640

Closed
wants to merge 3 commits into from
Closed

Security Update #1640

wants to merge 3 commits into from

Conversation

craidler-radconsulting
Copy link

Datatrans changed security settings so the sign needs to be sha256 with the Datatrans HMAC hex being converted to bin before using it to generate the sign itself.

@aschempp aschempp added the bug label Apr 8, 2016
@aschempp aschempp added this to the 2.3.5 milestone Apr 8, 2016
@aschempp
Copy link
Member

aschempp commented Apr 8, 2016

If we change this, it will affect existing installations, right?

@craidler-radconsulting
Copy link
Author

Deffo! So please feel free to change the push destination appropriatly or close PR.

@aschempp
Copy link
Member

aschempp commented Apr 8, 2016

can the user choose which hash algorithm to use in the Datatrans backend? Then we should probably provide the same options in Contao (similar as we do for Postfinance).

@craidler-radconsulting
Copy link
Author

Added select field for choosing the algorithm and the option to convert from hex2bin. Defaulted to md5 and 0, which will leave existing installation behaviour as is. I checked this on my system but i'd be happy if you could double check. Also i only added a german translation for it.

@Toflar
Copy link
Member

Toflar commented Apr 11, 2016

This should be tagged 2.4 then.

@aschempp aschempp added feature and removed bug labels Apr 27, 2016
@aschempp aschempp modified the milestones: 2.4.0, 2.3.5 Apr 27, 2016
@aschempp
Copy link
Member

aschempp commented Jul 6, 2016

Merged in 50a3f97

@aschempp aschempp closed this Jul 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants