https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
Christophe JAILLET changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
nsg-apache-httpd-maintena...@sophos.com changed:
What|Removed |Added
CC||nsg-apache-htt
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
--- Comment #19 from Yann Ylavic ---
Thanks Mika, it was more or less a cosmetic change which allowed me to close
the corresponding pull request on github (with the commit message, I missed
that on the first commits) :p
--
You are receiving t
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
--- Comment #18 from Micha Lenk ---
> Thanks for testing, backport proposed in r1830524.
The above mentioned change of the STATUS document lists another SVN revision
(r1830523). So I re-run our tests with that one applied on top. The issue
rem
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
--- Comment #17 from Yann Ylavic ---
Thanks for testing, backport proposed in r1830524.
--
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubsc
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
--- Comment #16 from Micha Lenk ---
I've just tested the cherry-picked SVN revisions r1829038 and r1829039 applied
to a httpd-2.4.25 build and can confirm that this variant of the patch also
fixes our issue.
Thanks for considering to backport
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
--- Comment #15 from Micha Lenk ---
>> Shouldn't pending_metadata get reset to zero within the if block?
>
> It's a local variable and the filter is leaving at this point.
Right. Thanks for the explanation.
--
You are receiving this mail bec
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
Yann Ylavic changed:
What|Removed |Added
Keywords|PatchAvailable |FixedInTrunk
--
You are receiving this
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
--- Comment #14 from Yann Ylavic ---
(In reply to Micha Lenk from comment #13)
> Yann, thanks for jumping on this so fast. I got something to test... :)
You're welcome. Fortunately you didn't test the series one by one :)
>
> One comment abou
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
--- Comment #13 from Micha Lenk ---
Yann, thanks for jumping on this so fast. I got something to test... :)
One comment about the current code in trunk (SVN r1829039), file
modules/filters/mod_xml2enc.c line 550. Shouldn't pending_metadata get
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
--- Comment #12 from Yann Ylavic ---
+ r1829039.
--
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: bugs-unsubscr...@httpd.ap
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
--- Comment #11 from Yann Ylavic ---
v5 committed in r1829038, will let it linger a bit for review before proposing
a backport to 2.4.x.
--
You are receiving this mail because:
You are the assignee for the bug.
---
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
Yann Ylavic changed:
What|Removed |Added
Attachment #35871|0 |1
is obsolete|
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
Yann Ylavic changed:
What|Removed |Added
Attachment #35870|0 |1
is obsolete|
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
Yann Ylavic changed:
What|Removed |Added
Attachment #35869|0 |1
is obsolete|
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
Yann Ylavic changed:
What|Removed |Added
Attachment #35770|0 |1
is obsolete|
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
--- Comment #6 from Yann Ylavic ---
I'd like the patch to aggregate meta buckets (besides FLUSH which can't be
retained), and then send them together either before further real data or
before leaving.
I didn't start working on this though, will
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
--- Comment #5 from Micha Lenk ---
Yann, is there any additional information needed from our side?
--
You are receiving this mail because:
You are the assignee for the bug.
-
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
--- Comment #4 from Eric Covener ---
Hi Yann, if you commit this please add "this closes #48" to the commit to close
out the github PR.
--
You are receiving this mail because:
You are the assignee for the bug.
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
--- Comment #3 from Micha Lenk ---
I've just tested your patch, and it fixes our issue with the dropped error
bucket too.
We didn't dare to do a bigger clean-up like you did, because we were not sure
what else could break by doing so. For that
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
--- Comment #2 from Yann Ylavic ---
Created attachment 35770
--> https://bz.apache.org/bugzilla/attachment.cgi?id=35770&action=edit
mod_xml2enc.c: pass all meta buckets down the chain
How about passing *all* metadata buckets down the chain (
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
Micha Lenk changed:
What|Removed |Added
Keywords||PatchAvailable
--
You are receiving this
https://bz.apache.org/bugzilla/show_bug.cgi?id=62180
Micha Lenk changed:
What|Removed |Added
CC||mi...@lenk.info
--- Comment #1 from Micha
23 matches
Mail list logo
