Merge pull request #39 from Sheldenburg/bug/fix-set-cookies

Sheldenburg · web-flow · commit b127eb5eaadd · 2024-10-23T06:43:39.000+11:00
test the domain param to fix the cookie error
diff --git a/backend/src/app/api/routes/login.py b/backend/src/app/api/routes/login.py
@@ -254,6 +254,7 @@ def github_oauth(session: SessionDep, code: str, response: Response):
             httponly=True,
             secure=True,
             samesite="lax",
+            domain="ai-engineer-template.vercel.app",
         )
         return response
 
@@ -282,5 +283,6 @@ def github_oauth(session: SessionDep, code: str, response: Response):
         httponly=True,
         secure=True,
         samesite="lax",
+        domain="ai-engineer-template.vercel.app",
     )
     return response

Original file line number	Diff line number	Diff line change
`@@ -254,6 +254,7 @@ def github_oauth(session: SessionDep, code: str, response: Response):`
`254`	`254`	`httponly=True,`
`255`	`255`	`secure=True,`
`256`	`256`	`samesite="lax",`
	`257`	`+ domain="ai-engineer-template.vercel.app",`
`257`	`258`	`)`
`258`	`259`	`return response`
`259`	`260`
`@@ -282,5 +283,6 @@ def github_oauth(session: SessionDep, code: str, response: Response):`
`282`	`283`	`httponly=True,`
`283`	`284`	`secure=True,`
`284`	`285`	`samesite="lax",`
	`286`	`+ domain="ai-engineer-template.vercel.app",`
`285`	`287`	`)`
`286`	`288`	`return response`