Skip to content
This repository was archived by the owner on Jan 11, 2023. It is now read-only.

this.fetch should { credentials: "include" } per default, if it's an internal resource. #1026

Closed
MatthiasGrandl opened this issue Dec 12, 2019 · 2 comments

Comments

@MatthiasGrandl
Copy link

On SSR the preload always threw an error and it took me quite a while to figure out, that it's because my credentials didn't get send for the auth middleware. I really think this should be the default for this.fetch on internal routes. Opinions?

@benmccann
Copy link
Member

Agree. It is the default on the browser. It's a bug that the server doesn't match the browser. There's a pending PR for it: #882

@Conduitry
Copy link
Member

Duplicate of #881, which has been fixed on master by #882.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants