Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Blocker 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.

        Activity

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development