ProActive Scheduler

In console interface, the print method arguments must not be formatted string

Details

  • Description:
    Hide
    Because we don't what is the message to display (it can be logs from user), the console must not use formatted string argument to write on the console.

    In some example, parenthesis in logs made the display crash because '(' and ')' characters are used to define group in formatted string.
    As we could not know the content of these strings, formatted string must be replaced by normal string.

    Two possible fixes are :
    - escape parenthesis chars in the concrete console,
    - change console API to prevent from giving formatted string
    Show
    Because we don't what is the message to display (it can be logs from user), the console must not use formatted string argument to write on the console. In some example, parenthesis in logs made the display crash because '(' and ')' characters are used to define group in formatted string. As we could not know the content of these strings, formatted string must be replaced by normal string. Two possible fixes are : - escape parenthesis chars in the concrete console, - change console API to prevent from giving formatted string

Activity

Hide
Jean-Luc Scheefer added a comment - 19/May/09 11:13 AM
The first fix "escape parenthesis chars in the concrete console" should be applied to version 1.0 for the version 1.0.1

The second one "change console API to prevent from giving formatted string " should be applied to version 1.0 for 1.1.
Show
Jean-Luc Scheefer added a comment - 19/May/09 11:13 AM The first fix "escape parenthesis chars in the concrete console" should be applied to version 1.0 for the version 1.0.1 The second one "change console API to prevent from giving formatted string " should be applied to version 1.0 for 1.1.
Hide
Jean-Luc Scheefer added a comment - 19/May/09 3:27 PM
"change console API to prevent from giving formatted string " fixed by Revision: 12704 for VERSION 1.1 ONLY
Show
Jean-Luc Scheefer added a comment - 19/May/09 3:27 PM "change console API to prevent from giving formatted string " fixed by Revision: 12704 for VERSION 1.1 ONLY
Hide
Jean-Luc Scheefer added a comment - 19/May/09 5:35 PM
Finally, parenthesis are not the problem, it comes from '%' character that must be escaped using %.
For a formatter, write "%%" to get a % at the output.

Anyway, the issue remains the same.
Show
Jean-Luc Scheefer added a comment - 19/May/09 5:35 PM Finally, parenthesis are not the problem, it comes from '%' character that must be escaped using %. For a formatter, write "%%" to get a % at the output. Anyway, the issue remains the same.
Hide
Jean-Luc Scheefer added a comment - 19/May/09 6:12 PM
Fixed by Revision: 12727 for VERSION 1.0.1 ONLY
Show
Jean-Luc Scheefer added a comment - 19/May/09 6:12 PM Fixed by Revision: 12727 for VERSION 1.0.1 ONLY

People

Dates

  • Created:
    19/May/09 11:12 AM
    Updated:
    04/Nov/09 5:01 PM
    Resolved:
    19/May/09 6:12 PM