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

Mollie stores not encrypted user bank account data within Magento #623

Closed
HenKun opened this issue Mar 16, 2023 · 2 comments
Closed

Mollie stores not encrypted user bank account data within Magento #623

HenKun opened this issue Mar 16, 2023 · 2 comments
Labels
Feature Request New feature or request

Comments

@HenKun
Copy link

HenKun commented Mar 16, 2023

Describe the bug
In sales_order_payment table in additional_information column, customer's bank account data is stored for SOFORT payments (and maybe others?) in an unencrypted way.

e.g.

"details":"{"consumerName":"XXX","consumerAccount":"DE52XXX","consumerBic":"SOLXXX"}"

I am not sure this is legal in all countries due to PSD2 things and privacy regulations.
Since it is not visible in frontend, module users might not be aware of this.
Even if it IS legal, if these information is not used anywhere in the system, it need not be stored imho.

Used versions

  • Magento Version number(eg 2.3.5): 2.4.2
  • Open source/Enterprise/B2b: OS
  • Mollie version number (Check configuration): 2.19.0

To Reproduce
Steps to reproduce the behavior:

  1. Create order
  2. Pay by "Sofort"
  3. Go to database into sales_order_payment
  4. See column additional_information

Expected behavior
Only private data is stored that is actually used or has a usecase.
If possible that required data should be encrypted.

Actual behavior
Bank account data is store in clear text.

@Frank-Magmodules
Copy link
Collaborator

Thank you for bringing up this matter. Although the module code itself may not require the storage of this information, it could potentially be utilized in downstream systems, such as direct refunds, that do not involve Mollie. 

While I acknowledge that this occurrence may not be ideal without the merchant's awareness, we plan to enhance the situation by implementing encryption by default. 

Additionally, we may provide an option for the merchant to restore the current behavior if they depend on it.

@Frank-Magmodules Frank-Magmodules added the Feature Request New feature or request label Mar 16, 2023
michielgerritsen added a commit to michielgerritsen/mollie-magento2 that referenced this issue Apr 11, 2023
@Frank-Magmodules
Copy link
Collaborator

Hello @HenKun,

We're pleased to inform you that the latest version of the Mollie plugin now includes the feature you requested. In the advanced section of the plugin, you will find an option to "Encrypt payment details" which you can enable to secure these details.

We hope this solution meets your requirements. For now, we will mark this issue as resolved. However, if you require further assistance, please feel free to reopen the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Request New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants