Commons Logging
  1. Commons Logging
  2. LOGGING-127

don't use setDefaultUseCaches() but setUseCaches() in LogFactory.getProperties()

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: Nightly Builds, 2.0
    • Fix Version/s: 1.1.2
    • Labels:
      None

      Description

      The fix for LOGGING-126 introduced a call to URLConnection.setDefaultUseCaches() in LogFactory.getProperties(). This call should be changed to URLConnection.setUseCaches(), because URLConnection.setDefaultUseCaches() sets a static variable whose value affects the whole system, while URLConnection.setUseCaches() only affects its target instance.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        5h 24m 1 Simon Kitching 20/Sep/08 17:40
        Resolved Resolved Closed Closed
        465d 32m 1 Dennis Lundberg 29/Dec/09 17:13
        Dennis Lundberg made changes -
        Fix Version/s 1.1.2 [ 12314498 ]
        Fix Version/s Nightly Builds [ 12311790 ]
        Dennis Lundberg made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Simon Kitching [ skitching ]
        Simon Kitching made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s Nightly Builds [ 12311790 ]
        Resolution Fixed [ 1 ]
        Hide
        Simon Kitching added a comment -

        Fix committed. Thanks for pointing that out!

        By the way: what a wierd API - a non-static method that controls a static flag.

        Show
        Simon Kitching added a comment - Fix committed. Thanks for pointing that out! By the way: what a wierd API - a non-static method that controls a static flag.
        Christopher Sahnwaldt created issue -

          People

          • Assignee:
            Simon Kitching
            Reporter:
            Christopher Sahnwaldt
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development