Re: Review Board Ticket #4438: sending emails fails with excessively large number of files

2017-05-26 Thread David Trowbridge
--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4438/
--

New update by torcolvin
For Beanbag, Inc. > Review Board > Ticket #4438


Reply:

Fixed in release-2.5.x (5e19619). This will ship in 2.5.12. Thanks!


Status:
- New
+ Fixed

-- 
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 https://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Re: Review Board Ticket #4438: sending emails fails with excessively large number of files

2017-01-20 Thread Barret Rennie
--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4438/
--

New update by torcolvin
For Beanbag, Inc. > Review Board > Ticket #4438


Assigned to:
+ DanielPBak

-- 
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 https://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Re: Review Board Ticket #4438: sending emails fails with excessively large number of files

2016-11-24 Thread Rick Liu
--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4438/
--

New update by torcolvin
For Beanbag, Inc. > Review Board > Ticket #4438


Reply:

I created a pull-request to disable "X-ReviewBoard-Diff-For" line to fix 
this issue:
https://github.com/reviewboard/reviewboard/pull/42

-- 
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 https://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Re: Review Board Ticket #4438: sending emails fails with excessively large number of files

2016-07-13 Thread David Trowbridge
--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4438/
--

New update by torcolvin
For Beanbag, Inc. > Review Board > Ticket #4438


Tags:
+ Component:EMail
+ EasyFix

-- 
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 https://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Re: Review Board Ticket #4438: sending emails fails with excessively large number of files

2016-07-12 Thread test test
--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4438/
--

New update by torcolvin
For Beanbag, Inc. > Review Board > Ticket #4438


Reply:

test

-- 
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 https://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Review Board Ticket #4438: sending emails fails with excessively large number of files

2016-07-02 Thread Tor Colvin
--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4438/
--

New ticket #4438 by torcolvin
For Beanbag, Inc. > Review Board

Status: New
Tags: Priority:Medium, Type:Defect


--
sending emails fails with excessively large number of files
==

# What version are you running?
2.5.6.1

# What steps will reproduce the problem?
(This problem is admittedly unlikely to happen with anything other than 
reviewing autogenerated code.)
1. File a review request with 400 files with file names with ~100 charcter names
2. Publish review request
3. Fail to receive email and note server log


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

The size is pathologically large since 
  File 
"/scr/rb/lib/python2.7/site-packages/reviewboard/notifications/email.py", line 
694, in send_review_mail
message.send()
  File "/scr/rb/lib/python2.7/site-packages/django/core/mail/message.py", line 
276, in send
return self.get_connection(fail_silently).send_messages([self])
  File "/scr/rb/lib/python2.7/site-packages/django/core/mail/backends/smtp.py", 
line 94, in send_messages
sent = self._send(message)
  File "/scr/rb/lib/python2.7/site-packages/django/core/mail/backends/smtp.py", 
line 110, in _send
self.connection.sendmail(from_email, recipients, message.as_bytes())
  File "/usr/lib64/python2.7/smtplib.py", line 720, in sendmail
raise SMTPDataError(code, resp)
SMTPDataError: (552, '5.6.0 Headers too large (32768 max)')

# What operating system are you using? What browser?

centos 7, sendmail

# Please provide any additional information below.

The header file is large since we add:

- X-ReviewBoard-Diff-For : full/path/to/filename

for each file

Obviously it's not possible for a human to meaningfully review all these files 
but the default MaxHeaderSize for sendmail on redhat systems is 32768 and may 
be different for other mail servers. It might make sense to truncate including 
filenames in headers of there is a large number of files.

--

-- 
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 https://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.