#1699 non-ascii charaters in milestone name i.e. ä will traceback
Closed: Fixed 7 years ago Opened 7 years ago by kordex.

I used 'käpy' as a test string for creating a milestone on repo. After issue is assigned with the milestone pagure will traceback:

pagure-2.10.1-1.fc24.noarch

mod_wsgi (pid=23270): Exception occurred processing WSGI script '/usr/share/pagure/pagure.wsgi'.
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/flask/app.py", line 1836, in __call__
    return self.wsgi_app(environ, start_response)
  File "/usr/lib/python2.7/site-packages/pagure/proxy.py", line 71, in __call__
    return self.app(environ, start_response)
  File "/usr/lib/python2.7/site-packages/flask/app.py", line 1820, in wsgi_app
    response = self.make_response(self.handle_exception(e))
  File "/usr/lib/python2.7/site-packages/flask/app.py", line 1403, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/lib/python2.7/site-packages/flask/app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/lib/python2.7/site-packages/flask/app.py", line 1477, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/lib/python2.7/site-packages/flask/app.py", line 1381, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/lib/python2.7/site-packages/flask/app.py", line 1475, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/lib/python2.7/site-packages/flask/app.py", line 1461, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/usr/lib/python2.7/site-packages/pagure/ui/issues.py", line 870, in view_issue
    form.milestone.data = str(issue.milestone)
UnicodeEncodeError: 'ascii' codec can't encode character u'\\xe4' in position 1: ordinal not in range(128)

@pingou Using issue.milestone.encode('utf-8') instead of str(issue.milestone) seems to solve this problem. Or is this issue more deep-seated than that?

@cep just dropping the str() worked for me, see PR #1707

@pingou :thumbsup: Can close this as fixed.

@cep well the PR hasn't been merged so this isn't fixed yet :)

@pingou changed the status to Closed

7 years ago

Login to comment on this ticket.

Metadata