Project

General

Profile

Actions

Bug #5676

closed

Add notifications to ncf-builder on success and error

Added by Benoît PECCATTE over 9 years ago. Updated almost 2 years ago.

Status:
Released
Priority:
2
Category:
Generic methods
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
Fix check:
Regression:

Description

When there is a session timeout on rudder and we are on an ncf builber page, we just get a blank page.

ncf builder should detect this (401) and automatically redirect to rudder authentication page.
To enable this, rudder should return its authentication url along with the error message when called from the api


Related issues 1 (0 open1 closed)

Related to Rudder - User story #5677: Rudder API should provide its authentication url on 401RejectedActions
Actions #1

Updated by François ARMAND about 9 years ago

  • Target version set to 2.11.8

To be more specific: this problem is particulary visible on Rudder 2.11, when ncf-builder is completly on an other page. In rudder 3.0 and up, we are forced to access a Rudder page, and so the authentication and session expiration are correctly handle by Rudder authentication schemes.

The first thing to do for that is to AT LEAST have an error message explaining to the user that he must authenticate.

Actions #2

Updated by François ARMAND about 9 years ago

  • Status changed from New to In progress
  • Assignee set to Vincent MEMBRÉ
  • Priority changed from N/A to 2
Actions #3

Updated by Vincent MEMBRÉ about 9 years ago

  • Target version changed from 2.11.8 to 0.x
Actions #4

Updated by Vincent MEMBRÉ about 9 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Vincent MEMBRÉ to François ARMAND
  • Pull Request set to https://github.com/Normation/ncf/pull/176
Actions #5

Updated by Vincent MEMBRÉ about 9 years ago

  • Subject changed from ncf builder doesn't cope well with rudder session timeout to Add notifications to ncf-builder on success and error
Actions #6

Updated by Vincent MEMBRÉ about 9 years ago

  • Status changed from Pending technical review to Pending release
  • % Done changed from 0 to 100

Applied in changeset commit:25f664950c563dcc1dddc19991a937ec8fd9b613.

Actions #7

Updated by François ARMAND about 9 years ago

Applied in changeset commit:e53b99939b5feb409d3e91dfa6cf7f038b6346ce.

Actions #8

Updated by Vincent MEMBRÉ over 7 years ago

  • Blocked by deleted (User story #5677: Rudder API should provide its authentication url on 401)
Actions #9

Updated by Vincent MEMBRÉ over 7 years ago

  • Related to User story #5677: Rudder API should provide its authentication url on 401 added
Actions #10

Updated by Vincent MEMBRÉ over 7 years ago

  • Status changed from Pending release to Released
Actions #11

Updated by Alexis Mousset almost 2 years ago

  • Target version changed from 0.x to ncf-0.x
  • Priority set to 0
Actions #12

Updated by Alexis Mousset almost 2 years ago

  • Project changed from 41 to Rudder
  • Category set to Generic methods
Actions

Also available in: Atom PDF