[error-report-web][PATCH] parser: Add Unicode support to the error log

[error-report-web][PATCH] parser: Add Unicode support to the error log

[mariano.lopez]

From: Mariano Lopez <mariano.lopez@linux.intel.com>

If the server receives a error log with Unicode
character in it, it will throw and internal server
error, this is caused because the server doesn't
try to convert everything to ASCII

This patch changes the log encoding to UTF-8 so
it will allow Unicode characters.

[YOCTO #8225]

Signed-off-by: Mariano Lopez <mariano.lopez@linux.intel.com>
---
 Post/parser.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Post/parser.py b/Post/parser.py
index d33a968..599afde 100644
--- a/Post/parser.py
+++ b/Post/parser.py
@@ -85,7 +85,7 @@ class Parser:
                 recipe = package
                 recipe_version = "unknown"
 
-            f = BuildFailure(TASK = str(fail['task']), RECIPE = recipe, RECIPE_VERSION = recipe_version, ERROR_DETAILS = str(fail['log']), BUILD = b)
+            f = BuildFailure(TASK = str(fail['task']), RECIPE = recipe, RECIPE_VERSION = recipe_version, ERROR_DETAILS = fail['log'].encode('utf-8'), BUILD = b)
 
             f.save()
 
-- 
1.9.1

Re: [error-report-web][PATCH] parser: Add Unicode support to the error log

[Michael Wood]

On 30/09/15 14:54, mariano.lopez@linux.intel.com wrote:
> From: Mariano Lopez <mariano.lopez@linux.intel.com>
>
> If the server receives a error log with Unicode
> character in it, it will throw and internal server
> error, this is caused because the server doesn't
> try to convert everything to ASCII
>
> This patch changes the log encoding to UTF-8 so
> it will allow Unicode characters.
>
> [YOCTO #8225]
>
> Signed-off-by: Mariano Lopez <mariano.lopez@linux.intel.com>
> ---
>   Post/parser.py | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Post/parser.py b/Post/parser.py
> index d33a968..599afde 100644
> --- a/Post/parser.py
> +++ b/Post/parser.py
> @@ -85,7 +85,7 @@ class Parser:
>                   recipe = package
>                   recipe_version = "unknown"
>   
> -            f = BuildFailure(TASK = str(fail['task']), RECIPE = recipe, RECIPE_VERSION = recipe_version, ERROR_DETAILS = str(fail['log']), BUILD = b)
> +            f = BuildFailure(TASK = str(fail['task']), RECIPE = recipe, RECIPE_VERSION = recipe_version, ERROR_DETAILS = fail['log'].encode('utf-8'), BUILD = b)
>   
>               f.save()
>   

Thanks for the patch. Now committed.

Michael