Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Cookie expire date can be integer or float#67

Merged
smnandre merged 1 commit intoplaywright-php:mainfrom
stloyd:bugfix/cookie-expire-as-float
Feb 25, 2026
Merged

Cookie expire date can be integer or float#67
smnandre merged 1 commit intoplaywright-php:mainfrom
stloyd:bugfix/cookie-expire-as-float

Conversation

@stloyd
Copy link
Contributor

@stloyd stloyd commented Feb 14, 2026

Refactor cookie validation into internal class to reduce duplication

@codecov
Copy link

codecov bot commented Feb 14, 2026

Codecov Report

❌ Patch coverage is 94.28571% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/Page/Page.php 0.00% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@stloyd stloyd force-pushed the bugfix/cookie-expire-as-float branch from ec0e5cd to dfcc5f3 Compare February 14, 2026 11:26
Refactor cookie validation into internal class to reduce duplication
@stloyd stloyd force-pushed the bugfix/cookie-expire-as-float branch from dfcc5f3 to 44f3ff4 Compare February 14, 2026 11:32
@stloyd
Copy link
Contributor Author

stloyd commented Feb 16, 2026

@smnandre Do you think this is overkill, or do you need more time to have a deeper look? Sorry if I sound harsh ;)

@smnandre
Copy link
Member

Please.... a contributor helping an OSS project... speak the way you want, this will never be harsh to me :)

@smnandre
Copy link
Member

Yeah I wanted to test it locally, and was busy working on the Symfony bundle this week-end ... I'm gonna look later tonight if that's ok for you ?

@smnandre smnandre merged commit 741f5c5 into playwright-php:main Feb 25, 2026
10 checks passed
@smnandre
Copy link
Member

Thank you very much @stloyd ! tried to reach you on X... you can contact me on slack or email if you want!

Anyway, thnk you very very much

@stloyd stloyd deleted the bugfix/cookie-expire-as-float branch February 25, 2026 20:19
@stloyd
Copy link
Contributor Author

stloyd commented Feb 25, 2026

I have limited visibility for messages from not followers, but I have same nick on Symfony slack if needed to reach me faster 😉
Anyway thanks for the merge 😉

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