Project

General

Profile

Actions

Bug #10067

closed

If an LDAP attribute is missing the related LDAP errors are not reported in UI (silent fail)

Added by François ARMAND about 7 years ago. Updated over 4 years ago.

Status:
Released
Priority:
N/A
Category:
Web - UI & UX
Target version:
Severity:
Major - prevents use of part of Rudder | no simple workaround
UX impact:
User visibility:
Infrequent - complex configurations | third party integrations
Effort required:
Very Small
Priority:
58
Name check:
Reviewed
Fix check:
Checked
Regression:

Description

The reported error is:

[2017-01-24 16:53:01] ERROR com.normation.ldap.sdk.RwLDAPConnection - Exception ignored (by configuration) when trying to modify entry 'directiveId=61d6bce5-116c-4e0b-85bf-93bdecc3eef5,activeTechniqueId=ab90f00a-d206-42dd-aa1b-9b493f2b4932,techniqueCategoryId=applications,techniqueCategoryId=Active Techniques,ou=Rudder,cn=rudder-configuration'.  Reported exception was: serializedTags: attribute type undefined
com.unboundid.ldap.sdk.LDAPException: serializedTags: attribute type undefined
    at com.unboundid.ldap.sdk.LDAPConnection.modify(LDAPConnection.java:2513) ~[unboundid-ldapsdk-2.3.6.jar:2.3.6]
    at com.normation.ldap.sdk.RwLDAPConnection$$anonfun$10$$anonfun$apply$7.apply(LDAPConnection.scala:541) ~[scala-ldap-4.1.0~beta2-SNAPSHOT.jar:na]
    at com.normation.ldap.sdk.RwLDAPConnection$$anonfun$10$$anonfun$apply$7.apply(LDAPConnection.scala:541) ~[scala-ldap-4.1.0~beta2-SNAPSHOT.jar:na]
    at scala.util.control.Exception$Catch.apply(Exception.scala:103) ~[na:na]
 ....

That error certainly should not be ignored.
It may happen since the start of (Rudder) times.

Actions #1

Updated by François ARMAND about 7 years ago

  • Subject changed from If LDAP attribute is missing, related LDAP error are not reported in UI (silent failt to If LDAP attribute is missing, related LDAP error are not reported in UI (silent fail)
  • Description updated (diff)
Actions #2

Updated by Benoît PECCATTE about 7 years ago

  • Category set to Web - UI & UX
Actions #3

Updated by François ARMAND about 7 years ago

  • Severity set to Major - prevents use of part of Rudder | no simple workaround
  • User visibility set to Infrequent - complex configurations | third party integrations
Actions #4

Updated by Benoît PECCATTE about 7 years ago

  • Priority set to 25
Actions #5

Updated by Benoît PECCATTE almost 7 years ago

  • Priority changed from 25 to 24
Actions #6

Updated by Benoît PECCATTE almost 7 years ago

  • Priority changed from 24 to 38
Actions #7

Updated by Benoît PECCATTE over 6 years ago

  • Priority changed from 38 to 36
Actions #8

Updated by François ARMAND almost 5 years ago

  • Assignee set to François ARMAND
  • Priority changed from 36 to 34
Actions #9

Updated by François ARMAND over 4 years ago

  • Effort required set to Very Small
  • Priority changed from 34 to 58
Actions #10

Updated by François ARMAND over 4 years ago

  • Target version set to 5.0.15
Actions #11

Updated by François ARMAND over 4 years ago

  • Status changed from New to In progress
Actions #12

Updated by François ARMAND over 4 years ago

This one is only reported with OpenLDAP, not with the test server in unboundID SDK.

Actions #13

Updated by François ARMAND over 4 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/rudder/pull/2583
Actions #14

Updated by François ARMAND over 4 years ago

OK, actually I'm not sure why we ignore most of the case set to be ignored by default here:

NO_SUCH_ATTRIBUTE => it's clearly a failure in all case, we are adding something not in the schem
UNDEFINED_ATTRIBUTE_TYPE => same
INVALID_ATTRIBUTE_SYNTAX => we are trying to add for ex a bool to a string, clearly an error.
INVALID_DN_SYNTAX => clearly an error we want to catch
ENCODING_ERROR => same.

Perhaps keep that one:

ATTRIBUTE_OR_VALUE_EXISTS, ENTRY_ALREADY_EXISTS => for idempotency

Actions #15

Updated by François ARMAND over 4 years ago

  • Status changed from Pending technical review to Pending release
Actions #16

Updated by Vincent MEMBRÉ over 4 years ago

  • Fix check set to To do
Actions #17

Updated by Vincent MEMBRÉ over 4 years ago

  • Name check set to To do
Actions #18

Updated by Alexis Mousset over 4 years ago

  • Subject changed from If LDAP attribute is missing, related LDAP error are not reported in UI (silent fail) to If an LDAP attribute is missing the related LDAP errors are not reported in UI (silent fail)
  • Name check changed from To do to Reviewed
Actions #19

Updated by François ARMAND over 4 years ago

  • Fix check changed from To do to Checked
Actions #20

Updated by Vincent MEMBRÉ over 4 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 5.0.15 which was released today.

Actions

Also available in: Atom PDF