express icon indicating copy to clipboard operation
express copied to clipboard

deps: [email protected]

Open rhodgkins opened this issue 4 months ago • 6 comments

To allow partitioned option to be used.

rhodgkins avatar Jan 23 '24 17:01 rhodgkins

Thank you @rhodgkins ! Would you be willing to add the new features this exposes to our test suite? A PR do our expressjs.com repo to add the new option would also be very appreciated.

dougwilson avatar Jan 24 '24 20:01 dougwilson

@dougwilson no problem, test added.

rhodgkins avatar Jan 29 '24 09:01 rhodgkins

And docs updated expressjs/expressjs.com#1456

rhodgkins avatar Jan 29 '24 09:01 rhodgkins

Sorry for the bump @dougwilson but is this ok now let me know if you need anything else. Thanks

rhodgkins avatar Feb 06 '24 14:02 rhodgkins

Any ETA of getting it merged? We need support of partitioned cookie in our project. @dougwilson

jitenderjayalwal avatar Feb 14 '24 06:02 jitenderjayalwal

Hello, the Express project is doing a bit of regrouping is all. We will begin to work on a new version roll out soon.

dougwilson avatar Feb 14 '24 10:02 dougwilson

Any luck with the progress on this, we cannot set auth cookies anymore, is there a workaround until this is complete?

1finedev avatar Mar 03 '24 18:03 1finedev

not trying to spam or to be a bother

I noticed this got some attention since yesterday, would this be merged anytime soon or is there a workaround, this is a huge pain point ATM

@dougwilson @UlisesGascon @sheplu

1finedev avatar Mar 04 '24 17:03 1finedev

Hey @1finedev, no worries about spamming, but also no need to ping individual contributors. Most of us are following all these repos for all activity, so the ping doesn't help. We are mostly done with the regrouping and landed our first release last week.

I will look at this one now, but my first question for @UlisesGascon is if there was a reason we did not land this before the last release. Was it just overlooked? I am unclear why we wouldn't have just merged if it was all good previously so just want to make sure we were not missing something.

wesleytodd avatar Mar 04 '24 17:03 wesleytodd

Hi again, please is there any chance this can be merged. Thanks

1finedev avatar Mar 13 '24 18:03 1finedev

Ah sorry I was supposed to reply here and forgot. So we will be working toward another release, but for the time being you can install the already published cookie package and use it instead of the bundled version from the express package itself.

wesleytodd avatar Mar 13 '24 19:03 wesleytodd

@UlisesGascon has this gone in any of the recent releases? I couldn't see anything in the release notes so thought I'd check...

rhodgkins avatar Mar 27 '24 14:03 rhodgkins

Ah ignore me, it's in History.md but wasn't in the notes on the release page.

Thanks 👍

rhodgkins avatar Mar 27 '24 14:03 rhodgkins