joe-blocher

Results 10 comments of joe-blocher

Maybe you can implement, like the 'ghost-admin-api-session': versions/5.54.4/core/server/services/auth/session/express-session.js ``` function getExpressSessionMiddleware() { ... name: 'ghost-admin-api-session', cookie: { maxAge: constants.SIX_MONTH_MS, httpOnly: true, path: urlUtils.getSubdir() + '/ghost', sameSite: urlUtils.isSSL(config.get('url')) ? 'none' :...

In core/frontend/apps/private-blogging/lib/middleware.js: ´´´ const privateBlogging = { … return session({ name: 'ghost-private', maxAge: constants.ONE_MONTH_MS, signed: false, // sameSite: 'none'

It's nearly three month later ... an nothing happened. But for me it closed, because I'm working locally :-) https://forum.ghost.org/t/make-this-site-private-not-working/39938

I have made my first PR. I think the 2. one failed: Merge branch 'main' into joe-blocher-patch-1 I don't really know what is for and how I can delete this...

**You did't fix the error: Make this site private - not working not working in Chrome and Opera** ![Bildschirmfoto 2024-04-18 um 10 34 22](https://github.com/TryGhost/Ghost/assets/93246850/e9d5d56b-4c6d-43af-9f51-ae2661ca01c0) ![Bildschirmfoto 2024-04-18 um 10 35 30](https://github.com/TryGhost/Ghost/assets/93246850/27bf768b-6695-4d3f-92a1-616c86af08ee)...

The pull request still not merged in version 5.82.2: [Fixed private mode cookie for local development #17938](https://github.com/TryGhost/Ghost/pull/17938) Why not?

I've downloaded the code: versions/5.82.2/core/frontend/apps/private-blogging/lib/middleware.js But the code is still the same: `return session({ name: 'ghost-private', maxAge: constants.ONE_MONTH_MS, signed: false, sameSite: 'none'

> > The code being the same does not mean your PR was not merged. In this case it looks like this change ended up possibly breaking something else so...

**Downloaded version 5.89.1 - this bug still not fixed** You have to change only 2 lines in your code. The first time I reported the solution in August 2023. Why...