Issue 1653 dropped content from web submissions to roundup

Title dropped content from web submissions to roundup
Priority not-our-bug Status given-up
Milestone Resolved in
Superseder Nosy List Serware, darcs-devel, dmitry.kurochkin, kowey, mornfall
Assigned To
Topics BugTracker

Created on 2009-10-22.07:42:51 by kowey, last changed 2017-07-30.23:38:55 by gh.

msg8991 (view) Author: kowey Date: 2009-10-22.07:42:44
I'm splitting this off from issue1596 (msg8737), because it's quite possible
that these are two separate bugs.

Sometimes roundup will seemingly randomly drop content from the 'Change Note'
when people submit something through the web interface.  We are totally at a
loss why and when this happens, but it's very frustrating.

It does not seem to operate under the same rules as the email submissions (which
drop content seemingly systematically from certain quoted lines and are
therefore a little better understood).

I need to submit this to the roundup project tracker.

We'd also need somebody to peruse the code and look for anything which might be
doing this sort of content manipulation
msg9003 (view) Author: kowey Date: 2009-10-22.13:52:18
So the related issue1596 was fixed by disabling a feature in roundup that trims
message bodies from emails.  But I'm going to temporarily re-enable it to test
out another hypothesis.

Maybe roundup is confused by lines that end in "-- " (Petr had tried "--", which
is close but not the syntax for email signatures).

If so, you should not see any more content after this line:-- 
and the fact that you're seeing this sentence means this hypothesis too is
msg9004 (view) Author: kowey Date: 2009-10-22.13:53:55
Nope, I guess that one is shot down too.
Huh. (another test, just that line alone)
msg9085 (view) Author: mornfall Date: 2009-10-30.11:58:51
We have another theory on how this may be coming around -- seems to be a
problem with word-wrapped lines, where the wrap happens between double dash and
an immediately following word (like, in middle of a commandline option). Let's
try now:

roundup seems to lose text, when it wraps this: --

msg9086 (view) Author: kowey Date: 2009-10-30.12:09:57
This is one very long line

Booh asoeih saeonti haneosi hasnteo hianteo shiaeosnti hsntueoa hisnteoai
hsntueoa hisnti hsnteoa hisntoai hnstioa hasnoti hnoa shinteo hisnteoeu --this
part of the line may get cut off!

Apparently, this is OK with IceWeasel and bad with Konqueror.  I'm very pleased
that we can reproduce this.  Let's try to narrow it down so we can give the
roundup guys a nice bug report.
msg9087 (view) Author: kowey Date: 2009-10-30.12:13:56
Trying again, this time with email and with a slightly longer line to
induce wrapping

Booh asoeih saeonti haneosi hasnteo hianteo shiaeosnti hsntueoa hisnteoai hsntueoa hisnti hsnteoa hisntoai hnstioa hasnoti hnoa shinteo hisnteoeu --wrapthis part of the line may get cut off!
msg9088 (view) Author: mornfall Date: 2009-10-30.12:14:57
Just a try with iceweasel (aka firefox):

msg9089 (view) Author: mornfall Date: 2009-10-30.16:20:03
This eventually turned out to be a KHTML bug in handling of WRAP=HARD textareas. Eric has worked around this on the roundup side by disabling the hard-wrapping for now. The KHTML developers have been informed of the issue.
msg9090 (view) Author: mornfall Date: 2009-10-30.16:21:05
(Hm, Eric, maybe you should afterall put WRAP=HARD back, because the rest of
the tracker seems to rely on this for sensible formatting. I will just try to
be careful now that I know what the problem is. Thanks.)
msg9091 (view) Author: kowey Date: 2009-10-30.16:27:39
OK, I've rolled back the patch in question (I guess not too many Konqueror users
will be bitten by it).

I guess we can mark this waiting-for since we're pretty much just waiting for
intervention from the rest of the universe (in this case, the KHTML team).  If
they ever fix this, we could probably do a test here.  Wont-fix is also fine if
we think that really no action on our part is needed.

Petr: a link to the relevant bug on their tracker might be handy.
Date User Action Args
2009-10-22 07:42:51koweycreate
2009-10-22 13:52:20koweysetnosy: kowey, darcs-devel, dmitry.kurochkin, Serware, mornfall
messages: + msg9003
2009-10-22 13:53:56koweysetnosy: kowey, darcs-devel, dmitry.kurochkin, Serware, mornfall
messages: + msg9004
2009-10-23 22:46:42adminsetnosy: + serware, - Serware
2009-10-23 23:32:06adminsetnosy: + Serware, - serware
2009-10-30 11:58:52mornfallsetmessages: + msg9085
2009-10-30 12:10:03koweysetmessages: + msg9086
2009-10-30 12:13:58koweysetmessages: + msg9087
2009-10-30 12:14:59mornfallsetmessages: + msg9088
2009-10-30 16:20:05mornfallsetstatus: needs-reproduction -> resolved
messages: + msg9089
2009-10-30 16:21:08mornfallsetstatus: resolved -> unknown
messages: + msg9090
2009-10-30 16:27:41koweysetpriority: bug -> not-our-bug
status: unknown -> waiting-for
messages: + msg9091
assignedto: kowey ->
2017-07-30 23:38:55ghsetstatus: waiting-for -> given-up