Uploaded image for project: 'Task Management'
  1. Task Management
  2. TASKMGMT-65

Remove dependency to cglib

    XMLWordPrintable

Details

    • Task
    • Resolution: Fixed
    • Neutral
    • 2.0.0
    • None
    • None

    Description

      The cglib library is not compatible with Java 17 and we need to get rid of it. As it comes in transitively through jackrabbit-ocm we can either replace or fix the latter:

      • Since jackrabbit-ocm is not supported anymore (last commit is 11 years old). Our safest bet is to replace it with an update technology. However, we need to think about how to maintain backward compatibility since we allow using the jackrabbit-ocm annotations in client code. See e.g. in the workflow module.
      • Probably quicker (but also dirtier) is to replace cglib with byte-buddy in jackrabbit-ocm (maybe even fork it!?).

      Implementation note

      Due to backward compatibility concerns (the jackrabbit-ocm annotations being exposed through our public API) we might consider fixing this in jackrabbit-ocm first while at the same time deprecating its usage on our side. In a next step we would reimplement the functionality and remove the deprecated jackrabbit-ocm one. This would have the advantage of fixing jackrabbit-ocm being relatively easy (replace cgLib with ByteBuddy) while at the same time giving us and our users some grace period.

      Discovery

      • This branch records some experiments performed by mgeljic for reimplementing TasksStore without Jackrabbit OCM. See also this discussion on Slack regarding forces in favour / against implementing a replacement on our side vs. fixing this in Jackrabbit OCM directly.
      • Since TASKMGMT-66, we have our own fork of jackrabbit-ocm. We can use that one to resolve this ticket.

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

                mduerig Michael Duerig
                mduerig Michael Duerig
                Foundation
                Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:
                  Work Started:

                  Checklists

                    Task DoR