[legacy-framework] Use cookie prefix from config.blitz.js instead of package.json name field

[mabadir]

Replaces: #1976
Closes: blitz-js/legacy-framework#329, blitz-js/legacy-framework#335

Adds new cookiePrefix property to sessionMiddleware config.

What are the changes and their implications?

module.exports = {
  middleware: [
    sessionMiddleware({
+     cookiePrefix: 'cookie',
      isAuthorized: simpleRolesIsAuthorized,
    }),
  ],
}

Feature Checklist

• Integration test added (see test docs if needed)
• Documentation added/updated (submit PR to blitzjs.com repo)

[mabadir]

Next step is to PR the docs for that change.

Also, I think we need to have some test coverage.

[flybayer]

Thanks! I fixed a bug where it wasn't working — example app test was failing.

A proper integration test can be added to test/integration/auth/ by setting the session cookie prefix and then adding a middleware to blitz.config.ts that checks and validates the res cookies. Probably should only do it on the first request? Or a certain request like login or something.

[flybayer]

Sweet, thanks Mina! I looked and the existing tests sufficiently cover this. I also just pushed a commit to docs canary to add this.

[blitzjs-bot]

Added @mabadir contributions for code