Skip to content

add GHSA-rx7m-68vc-ppxh#753

Merged
glaubinix merged 1 commit intoFriendsOfPHP:masterfrom
xabbuh:GHSA-rx7m-68vc-ppxh
Aug 29, 2025
Merged

add GHSA-rx7m-68vc-ppxh#753
glaubinix merged 1 commit intoFriendsOfPHP:masterfrom
xabbuh:GHSA-rx7m-68vc-ppxh

Conversation

@xabbuh
Copy link
Member

@xabbuh xabbuh commented Aug 29, 2025

No description provided.

@xabbuh xabbuh force-pushed the GHSA-rx7m-68vc-ppxh branch from 7d8bf00 to 421c2d2 Compare August 29, 2025 13:01
cve: CVE-2025-54370
branches:
1.x:
time: 2025-08-03 15:26:00
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I take it this here is the commit that fixed it? PHPOffice/PhpSpreadsheet@c2cd0e6

Screenshot 2025-08-29 at 15 12 20

The time looks off, wouldn't this be 01:26:00? AM instead of PM?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

according to GHSA-rx7m-68vc-ppxh I would say it's PHPOffice/PhpSpreadsheet@334a677 (but looks like I still messed up timezones, should be better now)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right, that's the fix on master but unless I am missing something that commit is still AM and not PM

Screenshot 2025-08-29 at 15 28 38

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

argh, you're right

@xabbuh xabbuh force-pushed the GHSA-rx7m-68vc-ppxh branch from 421c2d2 to 5fd02d8 Compare August 29, 2025 14:26
@xabbuh xabbuh force-pushed the GHSA-rx7m-68vc-ppxh branch from 5fd02d8 to 8d06636 Compare August 29, 2025 14:31
@glaubinix glaubinix merged commit 7159d3f into FriendsOfPHP:master Aug 29, 2025
1 check passed
@xabbuh xabbuh deleted the GHSA-rx7m-68vc-ppxh branch August 29, 2025 19:13
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 this pull request may close these issues.

2 participants