Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-16048

Examine Tika dependencies that brought in javax classes

    XMLWordPrintableJSON

Details

    Description

      SOLR-15989 Tika 1.28.1 upgrade brought in some javax packaged classes which janhoy caught with the smoketester. Details from janhoy in SOLR-15989:

      This upgrade adds many parsers and dependencies. I ran the smoketester, which introspects every jar looking for illegal "java." and "javax." class files. These libararies were flagged by the smoke tester:

      modules/extraction/lib/unit-api-1.0.jar" contains sheisty class "javax/measure/Dimension.class"
      modules/extraction/lib/jakarta.activation-1.2.2.jar" contains sheisty class "javax/activation/CommandInfo$Beans$1.class"
      modules/extraction/lib/jakarta.annotation-api-1.3.5.jar" contains sheisty class "javax/annotation/security/PermitAll.class"
      jakarta.xml.bind-api:2.3.3 

      I believes all jakarta libs are OK license wise, but strangely they use the javax.xxx namespace instead of jakarta.xxx. The Units API (javax.measure.xxx) seems to be in the same category, that it is a JSR implemented with an allowed license.

      We have excluded several annotation jars elsewhere, that's why I raise this here.

      I assume the correct action is to make exceptions for these pacakges in the smoke tester. Any other insight?

      Attachments

        Issue Links

          Activity

            People

              krisden Kevin Risden
              krisden Kevin Risden
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 1h
                  1h