httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Rainer Jung <rainer.j...@kippdata.de>
Subject Re: proxy API compat break in 2.4.7
Date Thu, 07 Apr 2016 20:04:58 GMT
Thanks, fully agreed. I added short notes in r1738184.

Regards,

Rainer

Am 07.04.2016 um 21:36 schrieb William A Rowe Jr:
> The first most important documentation of this should be in ap_mmn.h,
> as we've done in the past (you'll see some highlights).
>
> A note under the CHANGES entry for 2.4.7 seems worthwhile.
>
> Pointing this out under Announcement seems appropriate, that users
> of third party mod_proxy-consuming modules will need to recompile
> those modules when upgrading from 2.4.1 - 2.4.6.
>
> I can't think of another obvious place, other than any related doxygen.
>
> It seems to late to correct this situation, as half the world will benefit
> from leaving things as they are, and the other half of the world would
> have an easier time migrating from 2.4.early to 2.4.now.  Lose lose
> situation in my mind, so best to leave as-is, with docs.
>
>
>
> On Thu, Apr 7, 2016 at 1:09 PM, Rainer Jung <rainer.jung@kippdata.de
> <mailto:rainer.jung@kippdata.de>> wrote:
>
>     Revision 1560081 (backport of r1533087) introduced "char
>     uds_path[PROXY_WORKER_MAX_NAME_SIZE]" in the middle of the
>     proxy_worker_shared structure which IMHO broke compatibility between
>     2.4.6 and 2.4.7.
>
>     I noticed it just now, because I was implementing a custom LB method
>     and during runtime it always found s->status to be 0. I had compiled
>     it against an old 2.4.4 to ensure not using later features, but was
>     testing inside 2.4.18. Thus the offset of the status field
>     calculated for 2.4.4 actually pointed into the long uds_path string
>     which wasn't used and probably calloced with zeroes.
>
>     Any good idea how to best document the break for other module
>     developers?
>
>     Regards,
>
>     Rainer

Mime
View raw message