Uploaded image for project: 'Crowd'
  1. Crowd
  2. CWD-4379

Trying to store a custom attribute against an LDAP directory throws an ERROR in the logs

    XMLWordPrintable

    Details

    • Type: Suggestion
    • Status: Gathering Interest (View Workflow)
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Feedback Policy:

      Our product teams collect and evaluate feedback from a number of different sources. To learn more about how we use customer feedback in the planning process, check out our new feature policy.

      Description

      the LDAP directory has this method:

      @Override
          public void storeUserAttributes(final String username, final Map<String, Set<String>> attributes) throws UserNotFoundException, OperationFailedException
          {
              throw new OperationNotSupportedException("Custom user attributes are not yet supported for LDAP directories");
          }
      
      

      This bubbles up to updateUserLastActivity in TokenAuthenticationManagerImpl where it's caught and results in:

      catch (OperationFailedException e)
              {
                  //This is indicative of a more serious error so receives a higher priority warning
                  logger.error("Operation to update last active for user '{}' failed", username, e);
              }
      
      

      This problem would be better logged as a warning and without the stack trace.

      Workaround:

      Remove the modify user attributes permission from your directory configuration.  If you are using the modify user attributes functionality, it is possible to suppress the messages in the logs by adding the following to your <crowd install>/crowd-webapp/WEB-INF/classes/log4j.properties file: 

      log4j.logger.com.atlassian.crowd.manager.token.RecoveryModeAwareTokenAuthenticationManager=FATAL
      
      
      

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              ldally lachland
              Votes:
              6 Vote for this issue
              Watchers:
              10 Start watching this issue

                Dates

                Created:
                Updated: