Excessive logging with userids_enabled: false

We are running CF with UAA’s userids_enabled: false:

properties:
  uaa:
    scim:
      userids_enabled: false

With a log level of WARN, we find that the uaa.log is getting filled pretty fast with the following messages:

[2017-07-18 03:15:03.103] uaa - 22465 [http-nio-8443-exec-62] ....  WARN --- UserIdConversionEndpoints: Request from user cloud_controller_username_lookup received at disabled Id translation endpoint with filter:id …
[2017-07-18 03:15:03.104] uaa - 22465 [http-nio-8443-exec-62] .... ERROR --- ScimUserEndpoints: Unhandled exception in SCIM user endpoints.
org.cloudfoundry.identity.uaa.scim.exception.ScimException: Illegal operation.
        at org.cloudfoundry.identity.uaa.scim.endpoints.UserIdConversionEndpoints.findUsers(UserIdConversionEndpoints.java:101)
...

It appears that this comes from UserIdConversionEndpoints.java.

As userids_enabled seems to be a valid flag, we wonder if it is possible to cut down the noise a bit; especially the exception stack.

Thanks!

Steffen and Bernhard

[fhanik]

I agree that this is an issue. Expected errors should not be logged under ERROR or WARN

[shamus]

Thanks for the PR. Merged!