Details

    • Type: Bug
    • Status: Resolved
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 4.1.1
    • Fix Version/s: 4.1.2
    • Component/s: Framework, tapestry-core
    • Labels:
      None

      Description

      AssetService declares this field:

      static final DateFormat CACHED_FORMAT = new SimpleDateFormat("EEE, d MMM yyyy HH:mm:ss Z", Locale.ENGLISH);

      However, DateFormat objects are explicitly NOT thread safe. We've encountered this in our in-house concurrency testing. We'll have to patch our local 4.1.1 to fix this before deploying our application.

      a quick search did not find any other references to SimpleDateFormat that look like they could have the same issue.

      A simple CommonsPool object pool of date format objects would fix the problem.

        Attachments

          Activity

            People

            • Assignee:
              jkuhnert Jesse Kuhnert
              Reporter:
              gwoolsey Greg Woolsey
            • Votes:
              2 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: