Bug #18103

Errors when submitting external user groups not displayed

Added by Daniel Lobato Garcia 11 months ago. Updated 10 days ago.

Status:Closed
Priority:Normal
Assigned To:Daniel Lobato Garcia
Category:Authentication
Target version:Foreman - Team Marek Iteration 22
Difficulty: Bugzilla link:1404507
Found in release: Pull request:https://github.com/theforeman/foreman/pull/4902, https://github.com/theforeman/foreman/pull/4199
Story points-
Velocity based estimate-
Release1.17.0Release relationshipAuto

Description

When one submit an user group with external user groups, and this doesn't work for whatever reason, like:

Net::LDAP::Error - No route to host - connect(2)
LdapFluff::Generic::UnauthenticatedException
LdapFluff::ActiveDirectory::MemberService::UIDNotFoundException,

it should show these errors in the UI and ideally some text explaining how to solve the issue.
Currently the errors are merely logged to production.log, which leaves users unable to understand what happened.


Related issues

Related to Foreman - Bug #17992: 500 when external usergroup users don't match filter Closed 01/09/2017

Associated revisions

Revision 36f8e9c0
Added by Daniel Lobato Garcia about 1 month ago

Fixes #17992, #18103 - Improve external usergroup errors

When one submit an user group with external user groups, and this
doesn't work for whatever reason, like:

Net::LDAP::Error - No route to host - connect(2)
LdapFluff::Generic::UnauthenticatedException
LdapFluff::ActiveDirectory::MemberService::UIDNotFoundException,

it should show these errors in the UI and ideally some text explaining
how to solve the issue.

Currently the errors are merely logged to production.log, which leaves
users unable to understand what happened.

This commit makes those errors show up in the UI with a suggestion on
how to fix those we know

History

#1 Updated by Daniel Lobato Garcia 11 months ago

  • Related to Bug #17992: 500 when external usergroup users don't match filter added

#2 Updated by Daniel Lobato Garcia 11 months ago

17992 is a specific instance of this - this bug is for handling most cases, the other one for "users don't match LDAP filter but group does"

#3 Updated by Daniel Lobato Garcia 11 months ago

  • Bugzilla link set to 1404507

#4 Updated by The Foreman Bot 11 months ago

  • Status changed from New to Ready For Testing
  • Assigned To set to Daniel Lobato Garcia
  • Pull request https://github.com/theforeman/foreman/pull/4199 added

#5 Updated by Daniel Lobato Garcia 11 months ago

  • Target version set to Team Brad - Iteration 11

#6 Updated by Brad Buckingham 10 months ago

  • Target version deleted (Team Brad - Iteration 11)

#7 Updated by Daniel Lobato Garcia 10 months ago

  • Target version set to Team Daniel - Iteration 9

#8 Updated by Marek Hulán 7 months ago

  • Target version changed from Team Daniel - Iteration 9 to Team Marek backlog

#9 Updated by Dominik Hlavac Duran 7 months ago

  • Assigned To changed from Daniel Lobato Garcia to Dominik Hlavac Duran

#10 Updated by Dominik Hlavac Duran 7 months ago

  • Assigned To changed from Dominik Hlavac Duran to Daniel Lobato Garcia

#11 Updated by Marek Hulán 5 months ago

  • Target version changed from Team Marek backlog to Team Marek Iteration 18

#12 Updated by Marek Hulán 4 months ago

  • Target version changed from Team Marek Iteration 18 to Team Marek Iteration 19

#13 Updated by Marek Hulán 4 months ago

  • Target version changed from Team Marek Iteration 19 to Team Marek Iteration 20

#14 Updated by Marek Hulán 3 months ago

  • Target version changed from Team Marek Iteration 20 to Foreman - Team Marek Iteration 21

#15 Updated by Marek Hulán about 1 month ago

  • Target version changed from Foreman - Team Marek Iteration 21 to Foreman - Team Marek Iteration 22

#16 Updated by Tomer Brisker about 1 month ago

  • Release set to 1.17.0

#17 Updated by Anonymous about 1 month ago

  • % Done changed from 0 to 100
  • Status changed from Ready For Testing to Closed

#18 Updated by The Foreman Bot 10 days ago

  • Pull request https://github.com/theforeman/foreman/pull/4902 added

Also available in: Atom PDF