Support Cookies#44
Conversation
Fixes skevy#40. See this corresponding change in express-graphql: graphql/express-graphql#30
|
A word of warning: I haven't actually tested this change locally. I made it directly on GitHub, sorry about that. |
|
Thanks!! |
|
@brandonbloom Yeah, I'm needing this support too, but when I develop locally, it's not sending a 'Cookies' header to my GraphQL endpoint. See electron/electron#1606 for a bug with electron not supporting Cookies header. I've tried bundling https://www.npmjs.com/package/electron-cookies but doesn't seem to work. Ideas? |
|
I have found away around this by setting cookies in the mainWindow -- see kevinold@9cd8d6d I'll iterate and open a PR when I like the implementation. Right now you must set "Cookie" key then "URL!!!NAME!!!VAL" and it's a bit wonky. Any suggestions are welcome. |
|
For anyone curious, another alternative is to open your development site via |
Fixes #40.
See this corresponding change in express-graphql: graphql/express-graphql#30