From: Aline Manera <alinefm(a)br.ibm.com&gt; The error message and all its parameters must be string. So convert error code to string in order to accomplish it.

Signed-off-by: Aline Manera <alinefm(a)br.ibm.com&gt; --- src/kimchi/auth.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/kimchi/auth.py b/src/kimchi/auth.py index af3b610..b16f2db 100644 --- a/src/kimchi/auth.py +++ b/src/kimchi/auth.py @@ -107,7 +107,7 @@ def authenticate(username, password, service="passwd"): try: auth.authenticate() except PAM.error, (resp, code): - msg_args = {'userid': username, 'code': code} + msg_args = {'userid': username, 'code': str(code)} raise OperationFailed("KCHAUTH0001E", msg_args) return True

On 03/05/2014 09:49 AM, Shu Ming wrote: > 2014/3/5 3:35, Aline Manera: >> From: Aline Manera <alinefm(a)br.ibm.com&gt; >> >> The error message and all its parameters must be string. >> So convert error code to string in order to accomplish it. > > I think the code is str already. Why should it be converted > explicitly? Is there any error encountered? agree. Also python can converted the parameters well when format the string. This is different with C code. such as: In [218]: "This is a code test: %(code)s " % {"code": 2} Out[218]: 'This is a code test: 2 ' so is this is need? IMO, if we still need it to be converted explicitly. We can use %d, instead of %s as follow. In [221]: "This is a code test: %(code)d " % {"code": 2} Out[221]: 'This is a code test: 2 '

> >> >> Signed-off-by: Aline Manera <alinefm(a)br.ibm.com&gt; >> --- >> src/kimchi/auth.py | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/src/kimchi/auth.py b/src/kimchi/auth.py >> index af3b610..b16f2db 100644 >> --- a/src/kimchi/auth.py >> +++ b/src/kimchi/auth.py >> @@ -107,7 +107,7 @@ def authenticate(username, password, >> service="passwd"): >> try: >> auth.authenticate() >> except PAM.error, (resp, code): >> - msg_args = {'userid': username, 'code': code} >> + msg_args = {'userid': username, 'code': str(code)} >> raise OperationFailed("KCHAUTH0001E", msg_args) >> >> return True > > _______________________________________________ > Kimchi-devel mailing list > Kimchi-devel(a)ovirt.org > http://lists.ovirt.org/mailman/listinfo/kimchi-devel > > >

On 03/04/2014 10:49 PM, Shu Ming wrote: > 2014/3/5 3:35, Aline Manera: >> From: Aline Manera <alinefm(a)br.ibm.com&gt; >> >> The error message and all its parameters must be string. >> So convert error code to string in order to accomplish it. > > I think the code is str already. Why should it be converted > explicitly? Is there any error encountered? Yes. Take a look in github: https://github.com/kimchi-project/kimchi/issues/330 The error happens because KimchiException() converts all message and arguments to unicode So we need to ensure the arguments are strings or do it convert it on KimchiException() which I think can cause problems when a different type which does not have str representattion

> >> >> Signed-off-by: Aline Manera <alinefm(a)br.ibm.com&gt; >> --- >> src/kimchi/auth.py | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/src/kimchi/auth.py b/src/kimchi/auth.py >> index af3b610..b16f2db 100644 >> --- a/src/kimchi/auth.py >> +++ b/src/kimchi/auth.py >> @@ -107,7 +107,7 @@ def authenticate(username, password, >> service="passwd"): >> try: >> auth.authenticate() >> except PAM.error, (resp, code): >> - msg_args = {'userid': username, 'code': code} >> + msg_args = {'userid': username, 'code': str(code)} >> raise OperationFailed("KCHAUTH0001E", msg_args) >> >> return True > _______________________________________________ Kimchi-devel mailing list Kimchi-devel(a)ovirt.org http://lists.ovirt.org/mailman/listinfo/kimchi-devel

From: Aline Manera <alinefm(a)br.ibm.com&gt; The error message and all its parameters must be string. So convert error code to string in order to accomplish it. Signed-off-by: Aline Manera <alinefm(a)br.ibm.com&gt; --- src/kimchi/auth.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/kimchi/auth.py b/src/kimchi/auth.py index af3b610..b16f2db 100644 --- a/src/kimchi/auth.py +++ b/src/kimchi/auth.py @@ -107,7 +107,7 @@ def authenticate(username, password, service="passwd"): try: auth.authenticate() except PAM.error, (resp, code): - msg_args = {'userid': username, 'code': code} + msg_args = {'userid': username, 'code': str(code)} raise OperationFailed("KCHAUTH0001E", msg_args) return True

On 03/06/2014 10:53 PM, Aline Manera wrote: > On 03/06/2014 06:41 AM, Sheldon wrote: >> On 03/05/2014 03:35 AM, Aline Manera wrote: >>> From: Aline Manera <alinefm(a)br.ibm.com&gt; >>> >>> The error message and all its parameters must be string. >>> So convert error code to string in order to accomplish it. >>> >>> Signed-off-by: Aline Manera <alinefm(a)br.ibm.com&gt; >>> --- >>> src/kimchi/auth.py | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/src/kimchi/auth.py b/src/kimchi/auth.py >>> index af3b610..b16f2db 100644 >>> --- a/src/kimchi/auth.py >>> +++ b/src/kimchi/auth.py >>> @@ -107,7 +107,7 @@ def authenticate(username, password, >>> service="passwd"): >>> try: >>> auth.authenticate() >>> except PAM.error, (resp, code): >>> - msg_args = {'userid': username, 'code': code} >>> + msg_args = {'userid': username, 'code': str(code)} >>> raise OperationFailed("KCHAUTH0001E", msg_args) >>> >>> return True >> How about the follow change? >> It can avoid call str() before we raise KimchiException Everywhere. >> Also I think it is not good to other developers to call str >> explicitly Everywhere. >> >> diff --git a/src/kimchi/exception.py b/src/kimchi/exception.py >> index 71a4d11..73e88ce 100644 >> --- a/src/kimchi/exception.py >> +++ b/src/kimchi/exception.py >> @@ -50,7 +50,7 @@ class KimchiException(Exception): >> translation = gettext >> >> for key, value in args.iteritems(): >> - if not isinstance(value, unicode): >> + if isinstance(value, str): >> args[key] = unicode(value, 'utf-8') >> >> return unicode(translation.gettext(text), 'utf-8') % args >> >> >> > > What about always convert to string before to unicode? > > if not isinstance(value, unicode): > args[key] = unicode(*str*(value), 'utf-8') This is also OK. seems it is more understand to most python developer. actually it is same effect with the above code I suggested. AFAK, seems two types has no string representation, one is unicode and another is object derived from nothing.

The later object derived from nothing, we can consider it is a bug. Kimchi do not allowed this. class c1(): pass But I'm not worry about it. IMO, no one will not pass this instance of object to KimchiException as args. we just allow objects derived from python object class in kimchi, class c2(object): pass > > That way we guarantee everything will be unicode in the end. > -- Thanks and best regards! Sheldon Feng(???)&lt;shaohef(a)linux.vnet.ibm.com&gt; IBM Linux Technology Center _______________________________________________ Kimchi-devel mailing list Kimchi-devel(a)ovirt.org http://lists.ovirt.org/mailman/listinfo/kimchi-devel

On 03/06/2014 06:41 AM, Sheldon wrote: > On 03/05/2014 03:35 AM, Aline Manera wrote: >> From: Aline Manera <alinefm(a)br.ibm.com&gt; >> >> The error message and all its parameters must be string. >> So convert error code to string in order to accomplish it. >> >> Signed-off-by: Aline Manera <alinefm(a)br.ibm.com&gt; >> --- >> src/kimchi/auth.py | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/src/kimchi/auth.py b/src/kimchi/auth.py >> index af3b610..b16f2db 100644 >> --- a/src/kimchi/auth.py >> +++ b/src/kimchi/auth.py >> @@ -107,7 +107,7 @@ def authenticate(username, password, >> service="passwd"): >> try: >> auth.authenticate() >> except PAM.error, (resp, code): >> - msg_args = {'userid': username, 'code': code} >> + msg_args = {'userid': username, 'code': str(code)} >> raise OperationFailed("KCHAUTH0001E", msg_args) >> >> return True > How about the follow change? > It can avoid call str() before we raise KimchiException Everywhere. > Also I think it is not good to other developers to call str > explicitly Everywhere. > > diff --git a/src/kimchi/exception.py b/src/kimchi/exception.py > index 71a4d11..73e88ce 100644 > --- a/src/kimchi/exception.py > +++ b/src/kimchi/exception.py > @@ -50,7 +50,7 @@ class KimchiException(Exception): > translation = gettext > > for key, value in args.iteritems(): > - if not isinstance(value, unicode): > + if isinstance(value, str): > args[key] = unicode(value, 'utf-8') > > return unicode(translation.gettext(text), 'utf-8') % args > > > What about always convert to string before to unicode? if not isinstance(value, unicode): args[key] = unicode(*str*(value), 'utf-8')

That way we guarantee everything will be unicode in the end. _______________________________________________ Kimchi-devel mailing list Kimchi-devel(a)ovirt.org http://lists.ovirt.org/mailman/listinfo/kimchi-devel