Maven Doxia
  1. Maven Doxia
  2. DOXIA-498

PegDownProcessor parsing timeout makes builds non-deterministic.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 1.4
    • Fix Version/s: 1.5
    • Component/s: Module - Markdown
    • Labels:
      None
    • Environment:
      Linux, Oracle Java 1.7.0_25

      Description

      Parsing timeout given to PegDownProcessor should be set to Long.MAX_VALUE. The default value is 2000 (2 seconds) which means that builds of large sites will spuriously fail when built on slower computers.

      The parsing timeout is a security feature for use when offering Markdown parsing as a service taking user input, since this use case doesn't apply to a Maven build setting the timeout to Long.MAX_VALUE effectively disables the timeout.

        Activity

        Devin Reid created issue -
        Hide
        Kristian Rosenvold added a comment -

        Fixed in r1530991. Thanks for the report !

        Show
        Kristian Rosenvold added a comment - Fixed in r1530991. Thanks for the report !
        Kristian Rosenvold made changes -
        Field Original Value New Value
        Assignee Kristian Rosenvold [ krosenvold ]
        Fix Version/s 1.5 [ 19200 ]
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Closed [ 6 ]
        Mark Thomas made changes -
        Project Import Sun Apr 05 09:30:24 UTC 2015 [ 1428226224715 ]
        Mark Thomas made changes -
        Workflow jira [ 12719900 ] Default workflow, editable Closed status [ 12748522 ]
        Mark Thomas made changes -
        Project Import Sun Apr 05 23:17:25 UTC 2015 [ 1428275845026 ]
        Mark Thomas made changes -
        Workflow jira [ 12957346 ] Default workflow, editable Closed status [ 12994336 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Closed Closed
        38m 53s 1 Kristian Rosenvold 10/Oct/13 10:26

          People

          • Assignee:
            Kristian Rosenvold
            Reporter:
            Devin Reid
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development