Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.3
    • Fix Version/s: 1.1.0
    • Labels:
      None
    • Environment:

      Operating System: other
      Platform: Other

      Description

      Corrrected code from LogFactoryImpl is below. Log4j is only available if the
      the class loader that loads Log4JLogger can load Logger. The orginal test
      is incorrect when both of these conditions are met
      a.) the context loader can load Logger ( directly or not ), and
      b.) Log4jLogger is actually loaded by a parent of the loader that loads Logger.

      The failure of course occurs because Log4jLogger has a direct dep on Logger.

      protected boolean isLog4JAvailable() {

      try

      { /* <incorrect> */ //loadClass("org.apache.log4j.Logger"); //loadClass("org.apache.commons.logging.impl.Log4JLogger"); /* </incorrect> */ loadClass("org.apache.commons.logging.impl.Log4JLogger").getClassLoader().loadClass( "org.apache.log4j.Logger" ); return (true); }

      catch (Throwable t)

      { return (false); }

      }

        Activity

        Dennis Lundberg made changes -
        Fix Version/s 1.1.0 [ 12311848 ]
        Henri Yandell made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Henri Yandell made changes -
        Affects Version/s 1.0.3 [ 12311839 ]
        Henri Yandell made changes -
        Affects Version/s 1.0.3 [ 12311677 ]
        Component/s Logging [ 12311124 ]
        Key COM-1639 LOGGING-68
        Project Commons [ 12310458 ] Commons Logging [ 12310484 ]
        Assignee Jakarta Commons Developers Mailing List [ commons-dev@jakarta.apache.org ]
        Henri Yandell made changes -
        Field Original Value New Value
        issue.field.bugzillaimportkey 31597 12341791
        d created issue -

          People

          • Assignee:
            Unassigned
            Reporter:
            d
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development