Project

General

Profile

Actions

Bug #15867

closed

LoginCluster redirect broken with EnableBetaController: true

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

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
Story points:
-
Release relationship:
Auto

Description

So this note was precient https://dev.arvados.org/issues/15107#note-27

The login redirect generated by Controller passes an empty "remote" which causes the target cluster to fail:

https://172.17.0.2:8000/login?remote=&return_to=https%3A%2F%2F172.17.0.3%2Fusers%2Fwelcome%3Freturn_to%3D%252F
{"errors":["Invalid remote cluster id (req-1dbjeioi2ubya1rjphw3)"],"error_token":"1574438263+155fa05c"}

Either controller shouldn't do that, or the API server should accept that (or both).


Related issues

Related to Arvados - Story #15107: [controller] Implement native Google login (configurable as an alternative to sso-provider)ResolvedTom Clegg10/31/2019

Actions
Actions #1

Updated by Peter Amstutz about 3 years ago

  • Status changed from New to In Progress
Actions #2

Updated by Peter Amstutz about 3 years ago

  • Status changed from In Progress to New
Actions #3

Updated by Peter Amstutz about 3 years ago

  • Description updated (diff)
  • Assigned To set to Tom Clegg
Actions #4

Updated by Peter Amstutz about 3 years ago

  • Related to Task #15512: Review 15107-google-login added
Actions #5

Updated by Peter Amstutz about 3 years ago

  • Related to deleted (Task #15512: Review 15107-google-login)
Actions #6

Updated by Peter Amstutz about 3 years ago

  • Related to Story #15107: [controller] Implement native Google login (configurable as an alternative to sso-provider) added
Actions #7

Updated by Peter Amstutz about 3 years ago

  • Description updated (diff)
Actions #8

Updated by Tom Clegg about 3 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100
Actions #9

Updated by Peter Amstutz about 3 years ago

  • Release set to 22
Actions

Also available in: Atom PDF