Uploaded image for project: 'Nuxeo Platform'
  1. Nuxeo Platform
  2. NXP-23725

AuditLogger await should work with Stream impl

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 9.3
    • Fix Version/s: 9.10
    • Component/s: Audit

      Description

      The AuditLogger.await is used in Drive tests, the impl org.nuxeo.ecm.platform.audit.service.AbstractAuditBackend#await rely on the audit bulker, so it does not work with the new default stream audit writer.

      This must be fixed in the meantime Drive tests can use the old impl by disabling the stream audit writer, adding nuxeo.stream.audit.enabled = false in the nuxeo.conf.

      Also the Elasticsearch.WaitForIndexing operation should be extended to support audit, by adding a waitForAudit boolean parameter the operation should also wait for audit writer using AuditLogger.await and refresh the audit index.
      This can replace the Drive operation: NuxeoDrive.WaitForElasticsearchCompletion.

      The operation WaitForIndexing should be restricted to Adminsitrator.

        Attachments

          Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. NXP-25463 Server CMIS tests should wait for audit

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Task - A task that needs to be done. NXDRIVE-1048 Use the generic operation Elasticsearch.WaitForIndexing to wait for audit, following NXP-23725

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Task - A task that needs to be done. NXP-23762 Make NuxeoDrive.WaitForElasticsearchCompletion rely on Elasticsearch.WaitForIndexing

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. NXDRIVE-1043 Workaround tests failing systematically because of a wrong await on the stream audit

          • Major - Major loss of function.
          • Resolved

          New Feature - A new feature of the product, which has yet to be developed. NXP-22109 Provides an Audit writer impl with Computation

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved
          is required by

          Task - A task that needs to be done. NXDRIVE-1044 Remove temporary workaround for stream audit await added by NXDRIVE-1043

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

            Activity

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 6 hours
                  6h