Issue 3467 in reviewboard: Server error when viewing certain posted reviews

2014-07-06 Thread reviewboard

Status: New
Owner: 
Labels: Type-Defect Priority-Medium

New issue 3467 by sourto...@gmail.com: Server error when viewing certain  
posted reviews

http://code.google.com/p/reviewboard/issues/detail?id=3467

What version are you running?

2.0.2

What's the URL of the page containing the problem?

https://git.reviewboard.kde.org/r/112294/
(Please don't visit this excessively, we get an email for each internal  
error Reviewboard encounters)


What steps will reproduce the problem?
1. Post a diff which contains certain characters which the ASCII character  
set cannot encode.


What is the expected output? What do you see instead?

Normal diff view page.

What operating system are you using? What browser?

Issue is a server error. Using Python 2.7.3 on Linux.

Please provide any additional information below.

Relevant part of the backtrace:

   
File /usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.2-py2.7.egg/reviewboard/diffviewer/models.py,  
line 58, in recalculate_line_counts

files = tool.get_parser(self.binary).parse()

   
File /usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.2-py2.7.egg/reviewboard/scmtools/git.py,  
line 181, in parse

preamble += self.lines[i] + '\n'

UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 24:  
ordinal not in range(128)


===

Patching  
/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.2-py2.7.egg/reviewboard/scmtools/git.py  
as follows appears to fix the issue:


preamble += unicode(self.lines[i], 'utf-8') + '\n'

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Re: Issue 3309 in reviewboard: Really long comment causes unreadable review

2014-07-06 Thread reviewboard


Comment #5 on issue 3309 by chip...@gmail.com: Really long comment causes  
unreadable review

http://code.google.com/p/reviewboard/issues/detail?id=3309

Issue 3466 has been merged into this issue.

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.