Skip to content

feat: cookie authentication #246

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

Merged
merged 3 commits into from
Oct 26, 2022
Merged

feat: cookie authentication #246

merged 3 commits into from
Oct 26, 2022

Conversation

rescribet
Copy link
Contributor

@rescribet rescribet commented Oct 14, 2022

@joepio
Copy link
Member

joepio commented Oct 24, 2022

Custom checklist for PR (for some reason the template is missing in this PR):

edit: see OP

@rescribet rescribet force-pushed the cookie-authentication branch 2 times, most recently from 58e4726 to fce69d1 Compare October 24, 2022 11:46
@rescribet
Copy link
Contributor Author

You probably mean the changelog? That already has a new entry with the ticket number.

@rescribet
Copy link
Contributor Author

From what I can tell the test should succeed, but this PR isn't tested against that version of the server.

@joepio
Copy link
Member

joepio commented Oct 24, 2022

From what I can tell the test should succeed, but this PR isn't tested against that version of the server.

Yes we can ignore this E2E test, as this uses the master docker build. The rust PR E2E must succeed, though.

@rescribet rescribet force-pushed the cookie-authentication branch from fce69d1 to 4733819 Compare October 26, 2022 12:01
@joepio joepio mentioned this pull request Oct 26, 2022
3 tasks
@joepio joepio force-pushed the cookie-authentication branch from ad61fbc to f701476 Compare October 26, 2022 13:45
@rescribet rescribet force-pushed the cookie-authentication branch from f701476 to 24bd09d Compare October 26, 2022 14:09
@joepio joepio force-pushed the cookie-authentication branch from 24bd09d to cbab515 Compare October 26, 2022 14:56
@rescribet rescribet force-pushed the cookie-authentication branch from cbab515 to 76503da Compare October 26, 2022 15:05
@joepio joepio merged commit 8db4d33 into main Oct 26, 2022
@joepio joepio deleted the cookie-authentication branch October 26, 2022 16:01
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.

Cookie based auth
2 participants