515: Summaries over 300 characters cause a 500 error

HooHa*****@gmai***** (Google Code) (Is this you? Claim this profile.)
Aug. 27, 2008 
What's the URL of the page containing the problem?
post-review

What steps will reproduce the problem?
1. Create a ChangeList description with more than 500 characters before 
the first new line 
2. Submit the CL to review board using post-review
3. See the nasty error 500 page

What is the expected output? What do you see instead?
It appears that an SQL warning about truncating the summery is causing a 
script to crap out on its self
chipx86
#1 chipx86 
Can you attach the 500 page as an HTML file? Be sure to remove any confidential info
in there.
  • +NeedInfo
#2 HooHa*****@gmai***** (Google Code) (Is this you? Claim this profile.) 
Here it is
  • +
    error.html 
    <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
    <html lang="en">
    <head>
      <meta http-equiv="content-type" content="text/html; charset=utf-8">
      <meta name="robots" content="NONE,NOARCHIVE">
      <title>Warning at /api/json/reviewrequests/165/update_from_changenum/</title>
      <style type="text/css">
        html * { padding:0; margin:0; }
        body * { padding:10px 20px; }
        body * * { padding:0; }
        body { font:small sans-serif; }
        body>div { border-bottom:1px solid #ddd; }
        h1 { font-weight:normal; }
        h2 { margin-bottom:.8em; }
        h2 span { font-size:80%; color:#666; font-weight:normal; }
        h3 { margin:1em 0 .5em 0; }
        h4 { margin:0 0 .5em 0; font-weight: normal; }
        table { border:1px solid #ccc; border-collapse: collapse; width:100%; background:white; }
        tbody td, tbody th { vertical-align:top; padding:2px 3px; }
        thead th { padding:1px 6px 1px 3px; background:#fefefe; text-align:left; fon
chipx86
#3 chipx86
  • -NeedInfo
    +Accepted
  • +Component-Reviews
    +Milestone-Release1.0
  • +Summaries over 300 characters cause a 500 error
david
#4 david 
See also bug 438.
chipx86
#5 chipx86
  • +PendingReview
david
#6 david 
Fixed in SVN r1459.
  • -Accepted
    +Fixed