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: 3 comments 18 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

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

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