#3852 Git code browse of Zenko project is down [24706]

v1.0.0
closed
Cory Johns
support (446) sf-2 (994)
General
nobody
2015-08-20
2012-03-06
Chris Tsai
No

https://sourceforge.net/apps/trac/sourceforge/ticket/24706

Git code browse for Zenko project is down. http://sourceforge.net/p/zenko/git

It gives a 500 status code.

Git from command line is ok.

Thanks

Discussion

  • Dave Brondsema

    Dave Brondsema - 2012-03-06
    • Description has changed:

    Diff:

    --- old 
+++ new 
@@ -1,6 +1,6 @@
 https://sourceforge.net/apps/trac/sourceforge/ticket/24706

->Git code browse for Zenko project is down. http://sourceforge.net/projects/zenko/git
+>Git code browse for Zenko project is down. http://sourceforge.net/p/zenko/git

 >It gives a 500 status code.
    • milestone: limbo --> forge-mar-23
     

  • Dave Brondsema

    Dave Brondsema - 2012-03-06 
    Module /var/local/allura/Allura/allura/templates/jinja_master/lib.html:34 in template
<<    {% set title=h.really_unicode(title) %}
         {% if email %}
           <img src="{{g.gravatar(email, size=size)}}"
                alt="{{title or email}}"
                title="{{title or email}}"
>>  <img src="{{g.gravatar(email, size=size)}}"
Module allura.lib.gravatar:58 in url
<<      assert gravatar_id or email
           if gravatar_id is None:
               gravatar_id = id(email)
           if 'r' not in kw and 'rating' not in kw: kw['r'] = 'pg'
           return ('https://secure.gravatar.com/avatar/%s?%s' % (gravatar_id, urllib.urlencode(kw)))
>>  gravatar_id = id(email)
Module allura.lib.gravatar:17 in id
<<      if match:
               email = match.group(1)
           return hashlib.md5(email.strip().lower()).hexdigest()

       def url(email=None, gravatar_id=None, **kw):
>>  return hashlib.md5(email.strip().lower()).hexdigest()
UnicodeEncodeError: 'ascii' codec can't encode character u'\xed' in position 3: ordinal not in range(128)
     

  • Dave Brondsema

    Dave Brondsema - 2012-03-19
    • size: --> 2
     

  • Cory Johns - 2012-03-22
    • status: open --> in-progress
    • assigned_to: Cory Johns
     

  • Cory Johns - 2012-03-22
    • status: in-progress --> code-review
    • qa: Jenny Steele
     

  • Cory Johns - 2012-03-22

    allura:cj/3852

    During planning, we discussed fixing this both where the error occurred as well as "upstream." However, upon review, the value was already perfectly valid unicode, and it was just the implicit encoding using the default 'ascii' encoding instead of 'utf8' that was the problem.

    I added a test case, and I also forked the repo in question to my sandbox if you wish to verify there.

     

  • Jenny Steele - 2012-03-23
    • status: code-review --> closed
     

Log in to post a comment.