Skip to content

Set auth cookie for all allowed origins#81

Merged
feedmeapples merged 1 commit intomainfrom
cookies
Jan 3, 2022
Merged

Set auth cookie for all allowed origins#81
feedmeapples merged 1 commit intomainfrom
cookies

Conversation

@feedmeapples
Copy link
Copy Markdown
Contributor

@feedmeapples feedmeapples commented Jan 3, 2022

What was changed

Sets cookies for UI that is also served from a different & allowed origin

Why?

Making auth work with UI served from different origin

Checklist

  1. Closes

  2. How was this tested:

  • Start the ui-server and UI under different origins (ports work too)
  • Configure auth through development.yaml file
  • Navigate to UI and follow auth flow
  • Verify that auth cookie is set in browser dev tools
  1. Any docs updates needed?

@feedmeapples feedmeapples changed the title Set cookies for all allowed origins Set auth cookie for all allowed origins Jan 3, 2022
@feedmeapples feedmeapples merged commit f3c6778 into main Jan 3, 2022
@feedmeapples feedmeapples deleted the cookies branch January 3, 2022 22:10
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.

2 participants