Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: strict same site cookies non-json logging and dev docker compose #65

Merged
merged 1 commit into from
Sep 26, 2023

Conversation

jmcdo29
Copy link
Owner

@jmcdo29 jmcdo29 commented Sep 26, 2023

As I look at the docker logs directly, and not through something like grafana or prmetheus, it's better to keep them in the stream string format.

Cookies should now be set to SameSite=Strict to allow them to work properly in a production environment (hopefully).

There's now a docker-compose.dev.yml that immitates the docker-compose.prod.yml to allow for a "full test" of the system

As I look at the docker logs directly, and not through something like
grafana or prmetheus, it's better to keep them in the stream string format.

Cookies should now be set to SameSite=Strict to allow them to work properly
in a production environment (hopefully).

There's now a docker-compose.dev.yml that immitates the docker-compose.prod.yml
to allow for a "full test" of the system
@nx-cloud
Copy link

nx-cloud bot commented Sep 26, 2023

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 3c5fb00. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch


🟥 Failed Commands
nx affected -t e2e --parallel --configuration=ci
✅ Successfully ran 3 targets

Sent with 💌 from NxCloud.

@jmcdo29 jmcdo29 merged commit 31910ef into main Sep 26, 2023
7 checks passed
@jmcdo29 jmcdo29 deleted the fix/logging-and-login branch September 26, 2023 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant