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] Fixes for potential XSS in the Checkout, Address Book and Admin Panel #16241

Merged
merged 16 commits into from
May 10, 2024

Conversation

GSadee
Copy link
Member

@GSadee GSadee commented May 10, 2024

Q A
Branch? 1.12
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Related tickets
License MIT

This PR aims to solve 2 issues:

mpysiak and others added 16 commits May 9, 2024 07:15
…GSadee)

This PR was merged into the 1.12 branch.

Discussion
----------

| Q               | A
| --------------- | -----
| Branch?         | 1.12
| Bug fix?        | yes
| New feature?    | no
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | n/a
| License         | MIT

Fixes CVE-2024-29376. Reported here: https://github.com/r2tunes/Reports/blob/main/Sylius.md


Commits
-------

0a7fe9e Add js sanitizeInput function
89880cd Add sanitizer function to UIBundle
19cea9a Use function from UIBundle
3d66fb0 [AddressBook] Add scenario for preventing from a potential XSS attack
9255540 [Checkout] Add scenario for preventing from a potential XSS attack
30de6ff [Behat] Minor scenarios improvements after code review
This PR was merged into the 1.12 branch.

Discussion
----------

| Q               | A
|-----------------|-----
| Branch?         | 1.12
| Bug fix?        | yes
| New feature?    | no
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | n/a
| License         | MIT

Fixes potential xss in admin panel (Taxons and Products)

Commits
-------

d4812f9 Fix potential xss in admin panel
29d18a3 Use function from UIBundle
c11c424 Fix product-auto-complete
a17de6d Test adding new taxon
d25edf3 Test adding new simple product
63c3cf7 Test adding similar products
679e793 Fixes after CR
@GSadee GSadee added Critical Issues and PRs, which are critical and should be fixed ASAP. Admin AdminBundle related issues and PRs. Shop ShopBundle related issues and PRs. Bug Confirmed bugs or bugfixes. labels May 10, 2024
@GSadee GSadee requested review from a team as code owners May 10, 2024 07:46
@TheMilek TheMilek merged commit fb0ecb2 into Sylius:1.12 May 10, 2024
27 checks passed
@TheMilek
Copy link
Member

Thank you, @GSadee!

Copy link

github-actions bot commented May 10, 2024

Bunnyshell Preview Environment deployment failed

Check https://github.com/Sylius/Sylius/actions/runs/9029035698 for details.

Available commands:

  • /bns:deploy to redeploy the environment
  • /bns:delete to remove the environment

@GSadee GSadee deleted the potential-xss-fixes branch May 10, 2024 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin AdminBundle related issues and PRs. Bug Confirmed bugs or bugfixes. Critical Issues and PRs, which are critical and should be fixed ASAP. Shop ShopBundle related issues and PRs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants