Skip to content

[Question]: Require login every time refresh #572

Discussion options

You must be logged in to vote

I know what's going on now after thinking about this more. It's related to this: #471

@fuegovic @anes300 @SailFlorve

The cookies are set to secure in "production" mode, which is the default when you run the server. According to expressjs docs:

secure - Ensures the browser only sends the cookie over HTTPS.

I forget this because they are still sent in a localhost environment. Without cookies, you need to login every time.

After I merge this PR, you can run the cookies in the insecure mode by running npm run backend:dev to start the app. For docker, you change the NODE_ENV in the compose file to development

Related PR: #1088

Please confirm this works

Replies: 4 comments 19 replies

Comment options

You must be logged in to vote
12 replies
@SailFlorve
Comment options

@danny-avila
Comment options

@anes300
Comment options

@SailFlorve
Comment options

@fuegovic
Comment options

Comment options

You must be logged in to vote
4 replies
@davecrab
Comment options

@danny-avila
Comment options

@davecrab
Comment options

@itoudium
Comment options

Answer selected by danny-avila
Comment options

You must be logged in to vote
2 replies
@fuegovic
Comment options

@snab43
Comment options

Comment options

You must be logged in to vote
1 reply
@AumCoin
Comment options

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Converted from issue

This discussion was converted from issue #570 on July 03, 2023 05:05.