Uploaded image for project: 'Kuali Rice Development'
  1. Kuali Rice Development
  2. KULRICE-3422

Reduce KIM logging output amount, set default log levels so permission checks are not logged

    Details

    • Type: Task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.0
    • Fix Version/s: 1.0
    • Component/s: Development
    • Labels:
      None

      Description

      Change the permission check logging so that stack unwinding only occurs if levels are set to TRACE. Also, reduce the default log levels so that permission check logging isn't shown.

        Attachments

          Activity

          Hide
          gilesp Peter Giles (Inactive) added a comment -

          IdentityManagementServiceImpl.java:
          ResponsibilityServiceImpl.java:
          RoleManagementServiceImpl.java:
          RoleRouteModule.java: (not a kim service, but I hit it too)

          • Refactored logging methods so that stack unwinding is only done when TRACE is enabled.
          • Moved from StringBuffer to StringBuilder to avoid unneeded synchronization

          impl/src/main/resources/org/kuali/rice/core/logging/default-log4j.properties:

          • Commented out lines that set kim services to DEBUG

          web/src/main/config/example-config/log4j.properties:

          • Commented out lines that set kim services to DEBUG
          • Added custom log format as per KULRICE-3237/KFSMI-4055 (I'd missed this file)
          Show
          gilesp Peter Giles (Inactive) added a comment - IdentityManagementServiceImpl.java: ResponsibilityServiceImpl.java: RoleManagementServiceImpl.java: RoleRouteModule.java: (not a kim service, but I hit it too) Refactored logging methods so that stack unwinding is only done when TRACE is enabled. Moved from StringBuffer to StringBuilder to avoid unneeded synchronization impl/src/main/resources/org/kuali/rice/core/logging/default-log4j.properties: Commented out lines that set kim services to DEBUG web/src/main/config/example-config/log4j.properties: Commented out lines that set kim services to DEBUG Added custom log format as per KULRICE-3237 / KFSMI-4055 (I'd missed this file)
          Hide
          ewestfal Eric Westfall added a comment -

          Bulk change of all Rice 1.0 issues to closed after public release.

          Show
          ewestfal Eric Westfall added a comment - Bulk change of all Rice 1.0 issues to closed after public release.

            People

            • Assignee:
              gilesp Peter Giles (Inactive)
              Reporter:
              gilesp Peter Giles (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: