Uploaded image for project: 'Grouper'
  1. Grouper
  2. GRP-1855

grouperPatchStatusFile can't read patch file in a jar

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 2.4.0
    • Fix Version/s: None
    • Component/s: API, UI
    • Labels:
      None

      Description

      When the UI starts (this is in the startup, so may affect other modules), it logs error:

      2018-08-01 15:20:50,413: [default task-2] ERROR GrouperVersion.patchesInstalled(440) - Error: cant process grouperPatchStatus.properties
      java.lang.RuntimeException: log4j.properties doesnt exist
      at edu.internet2.middleware.grouper.misc.GrouperVersion.grouperPatchStatusFile(GrouperVersion.java:379)
      at edu.internet2.middleware.grouper.misc.GrouperVersion.patchesInstalled(GrouperVersion.java:405)
      at edu.internet2.middleware.grouper.misc.GrouperStartup.appendPatchLevelsToStartupString(GrouperStartup.java:203)
      at edu.internet2.middleware.grouper.misc.GrouperStartup.printConfigOnce(GrouperStartup.java:137)
      at edu.internet2.middleware.grouper.misc.GrouperStartup.startup(GrouperStartup.java:271)
      at edu.internet2.middleware.grouper.subj.SubjectResolverFactory.getInstance(SubjectResolverFactory.java:58)
      at edu.internet2.middleware.grouper.SubjectFinder.getResolver(SubjectFinder.java:928)
      at edu.internet2.middleware.grouper.SubjectFinder.findRootSubject(SubjectFinder.java:913)
      at edu.internet2.middleware.grouper.GrouperSession.startRootSession(GrouperSession.java:427)
      at edu.internet2.middleware.grouper.GrouperSession.startRootSession(GrouperSession.java:445)
      at edu.internet2.middleware.grouper.j2ee.status.GrouperStatusServlet.doGet(GrouperStatusServlet.java:166)
      at javax.servlet.http.HttpServlet.service(HttpServlet.java:687)

      This is a non-fatal error. In our installation, our log4j.properties file is embedded in an included jar file. The GrouperVersion.grouperPatchStatusFile() assumes these are filesystem objects, so can't read them from other types of containers.

      This may need the same change as for the older subject.properties change – just change from files and paths to resources and uri's.

        Smart Checklist

          Attachments

            Activity

              People

              • Assignee:
                chris.hyzer@at.internet2.edu Chris Hyzer (upenn.edu)
                Reporter:
                chad.redman@at.internet2.edu Chad Redman (unc.edu)
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated: