GHSA-25fx-mxc2-76g7
Suggest an improvement- Source
- https://github.com/advisories/GHSA-25fx-mxc2-76g7
- Import Source
- https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2021/10/GHSA-25fx-mxc2-76g7/GHSA-25fx-mxc2-76g7.json
- JSON Data
- https://api.osv.dev/v1/vulns/GHSA-25fx-mxc2-76g7
- Aliases
- Published
- 2021-10-06T17:49:11Z
- Modified
- 2023-11-08T04:06:51.791756Z
- Severity
-
- 7.5 (High) CVSS_V3 - CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:N/A:N CVSS Calculator
- Summary
- Sylius PayPal Plugin allows unauthorized access to Credit card form, exposing payer name and not requiring 3DS
- Details
-
Impact
URL to the payment page done after checkout was created with autoincremented payment id (
/pay-with-paypal/{id}) and therefore it was easy to access for anyone, not even the order's customer. The problem was, the Credit card form has prefilled "credit card holder" field with the Customer's first and last name. Additionally, the mentioned form did not require a 3D Secure authentication, as well as did not checked the result of the 3D Secure authentication.Patches
The problem has been patched in Sylius/PayPalPlugin 1.2.4 and 1.3.1
Workarounds
One can override a
sylius_paypal_plugin_pay_with_paypal_formroute and change its URL parameters to (for example){orderToken}/{paymentId}, then override theSylius\PayPalPlugin\Controller\PayWithPayPalFormActionservice, to operate on the payment taken from the repository by these 2 values. It would also require usage of custom repository method. Additionally, one could override the@SyliusPayPalPlugin/payWithPaypal.html.twigtemplate, to addcontingencies: ['SCA_ALWAYS']line inhostedFields.submit(...)function call (line 421). It would then have to be handled in the function callback.For more information
If you have any questions or comments about this advisory: - Open an issue in Sylius/PayPalPlugin issues - Email us at security at sylius dot com
- Database specific
{ "nvd_published_at": "2021-10-05T21:15:00Z", "github_reviewed_at": "2021-10-05T21:12:27Z", "severity": "HIGH", "github_reviewed": true, "cwe_ids": [ "CWE-200", "CWE-639" ] }- References
-
- https://github.com/Sylius/PayPalPlugin/security/advisories/GHSA-25fx-mxc2-76g7
- https://nvd.nist.gov/vuln/detail/CVE-2021-41120
- https://github.com/Sylius/PayPalPlugin/commit/2adc46be2764ccee22b4247139b8056fb8d1afff
- https://github.com/Sylius/PayPalPlugin/commit/814923c2e9d97fe6279dcee866c34ced3d2fb7a7
- https://github.com/Sylius/PayPalPlugin
Affected packages
Packagist / sylius/paypal-plugin
Package
- Name
- sylius/paypal-plugin
- Purl
- pkg:composer/sylius/paypal-plugin
Packagist / sylius/paypal-plugin
Package
- Name
- sylius/paypal-plugin
- Purl
- pkg:composer/sylius/paypal-plugin