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.

Sign up for GitHub

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

Jump to bottom

test logout and login #390

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

test logout and login #390

wants to merge 10 commits into from

Conversation

anjalakaturi
Copy link
Contributor

@anjalakaturi anjalakaturi commented Apr 11, 2023 •
edited
Loading

closes #384

@anjalakaturi anjalakaturi changed the title test logout and login - need for admin tests test logout and login May 4, 2023
@danielfang97
Copy link
Contributor

I am not sure how logout is supposed to work, but I don't think this url is valid for SBH3? url: http://localhost:6789/login?logout
Also, /logout is bugged in SBH3, so the usable endpoint is /do_logout for the time being. Would need to be addressed later.

@cjmyers cjmyers added this to the SBH 3.0.0 milestone Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danielfang97 danielfang97 Awaiting requested review from danielfang97

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
SBH 3.0.0
Development

Successfully merging this pull request may close these issues.

/logout doesn't work on SBH3
3 participants
@anjalakaturi @danielfang97 @cjmyers