Project

General

Profile

Actions

Bug #16101

closed

https://ce8i5.arvadosapi.com/users/sign_out path not found

Added by Peter Amstutz about 4 years ago. Updated about 4 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Login
Target version:
Story points:
-
Release relationship:
Auto

Description

When using Controller's native Google login, logging out results in an error.


Subtasks 1 (0 open1 closed)

Task #16149: Review 16101-logoutResolvedPeter Amstutz02/18/2020Actions
Actions #1

Updated by Peter Amstutz about 4 years ago

  • Description updated (diff)
Actions #2

Updated by Peter Amstutz about 4 years ago

  • Category set to Login
Actions #3

Updated by Peter Amstutz about 4 years ago

  • Target version changed from 2020-02-12 Sprint to 2020-02-26 Sprint
Actions #4

Updated by Tom Clegg about 4 years ago

Controller needs a Logout() method that bypasses railsProxy when Login.GoogleClientID is configured, similar to Login() in source:lib/controller/localdb/conn.go. The native implementation doesn't set cookies so I think this is a no-op, just redirecting to the return_to param value (or if none provided, workbench2?).

Actions #5

Updated by Peter Amstutz about 4 years ago

  • Target version changed from 2020-02-26 Sprint to 2020-03-11 Sprint
Actions #6

Updated by Peter Amstutz about 4 years ago

  • Target version changed from 2020-03-11 Sprint to 2020-02-26 Sprint
Actions #7

Updated by Peter Amstutz about 4 years ago

  • Assigned To set to Tom Clegg
Actions #8

Updated by Tom Clegg about 4 years ago

  • Status changed from New to In Progress
Actions #10

Updated by Peter Amstutz about 4 years ago

Tom Clegg wrote:

16101-logout @ e459f4e2d40762f67ffedafbe988c8da6f4f04d4 -- developer-run-tests: #1736

It looks like if return_to is unspecified you'll get a blank (invalid) redirect. The API server sends you to the API server root (return_to = params[:return_to] || root_url) but sending you to Workbench by default would probably be better.

Actions #12

Updated by Peter Amstutz about 4 years ago

Tom Clegg wrote:

16101-logout @ e15c73164eaedc121420b328a7b26da3c35bc145 -- developer-run-tests: #1738

This LGTM

Actions #13

Updated by Anonymous about 4 years ago

  • % Done changed from 0 to 100
  • Status changed from In Progress to Resolved
Actions #14

Updated by Peter Amstutz about 4 years ago

  • Release set to 29
Actions

Also available in: Atom PDF