An open_entity without a close_entity call is pointless, those can
be resumed to a single call.
Index: trunk/modules/cache/cache_storage.c
===
--- trunk.orig/modules/cache/cache_storage.c
+++ trunk/modules/cache/cache_storage.c
@@ -40
Move parts of cache_select() to a smaller cache_check_request()
function that is easier to read and understand.
Index: trunk/modules/cache/cache_storage.c
===
--- trunk.orig/modules/cache/cache_storage.c
+++ trunk/modules/cache/cache_
Move the open_entity call to cache_recall_entity_headers, adding a
new urlkey parameter.
Index: trunk/modules/cache/cache_storage.c
===
--- trunk.orig/modules/cache/cache_storage.c
+++ trunk/modules/cache/cache_storage.c
@@ -37,8 +37,
Later on those will handle all HTTP specific caching nuances
Index: trunk/modules/cache/cache_storage.c
===
--- trunk.orig/modules/cache/cache_storage.c
+++ trunk/modules/cache/cache_storage.c
@@ -24,6 +24,30 @@ extern module AP_MODUL
Move parts of cache_url_handler() to a smaller add_cache_filters()
function that is easier to read and understand.
Index: trunk/modules/cache/mod_cache.c
===
--- trunk.orig/modules/cache/mod_cache.c
+++ trunk/modules/cache/mod_cache.c
Don't pass 'large' objects by value when not needed, it wastes time and space.
Index: trunk/modules/cache/cache_util.c
===
--- trunk.orig/modules/cache/cache_util.c
+++ trunk/modules/cache/cache_util.c
@@ -28,40 +28,41 @@ extern modul
Move parts of cache_save_filter() to a smaller check_cacheable_request()
function that is easier to read and understand. Also, a useless assignment
is removed.
Index: trunk/modules/cache/mod_cache.c
===
--- trunk.orig/modules/cache/m
According to my reading of RFC3986 (section 6.2.3.) if a URI contains an
authority component and an empty path, the empty path is to be equivalent
to "/".
It explicitly cites the following four URIs as equivalents:
http://example.com
http://example.com/
http://example.com:/
http:/
Don't cache requests with a expires date in the past; otherwise the cache code
will _always_ try to cache the URL, leading to numerous rename() errors on win32
since the URL is already cached.
Index: trunk/modules/cache/mod_cache.c
==
This patch converts the mod_disk_cache cache directory structure to a
uniformly distributed N level hierarchy. The admin specifies the number
of levels (or directories) and the files are scattered across the
directories. Example:
CacheRoot /cache/
# CacheDirLevels n l1 l2 ln
CacheDirLevels 2 4 256
Hi,
Here is the first code drop regarding my SoC proposal "Redesigning and extending
the Apache cache architecture" (http://verdesmares.com/Apache/proposal.txt).
The patches are mainly clenaups and bug-fixes, except a new mod_disk_cache
directory structure. Further clenaups still required.
Other
Hello Nick,
> What I'd like to see is dynamic loading of apr-util modules, and a
> build process as simple as apxs. But that's another subject.
I agree, they are both important features to stimulate wider deployment and
courage users to write extensions for this relatively new module.
> Tha
+---+
| Bugzilla Bug ID |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned
+---+
| Bugzilla Bug ID |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned
On 07/23/2006 11:56 PM, Nick Kew wrote:
> No. ap_dbd_prepare is about preparing statements from httpd.conf.
> In request processing, you'd acquire a connection and use apr_dbd_*.
Thanks for clarification.
Regards
Rüdiger
Ruediger Pluem wrote:
Sounds like a good idea to me to work this either into the setaside code on
apr-util side
or into ap_save_brigade
++1 to both...
ap_save_brigade_ex ?
For setaside, of course, we could have a setaside_ex member, but it seems
to me we can have a common struct that persis
On Sunday 23 July 2006 22:53, Ruediger Pluem wrote:
> On 07/23/2006 11:41 PM, Nick Kew wrote:
> > Not relevant. It's only used to store definitions until post_config.
> > Note that dbd_prepared_defns itself is allocated on ptemp, too.
>
> Just for my clarification: ap_dbd_prepare is not called *af
On 07/23/2006 11:41 PM, Nick Kew wrote:
>
> Not relevant. It's only used to store definitions until post_config.
> Note that dbd_prepared_defns itself is allocated on ptemp, too.
Just for my clarification: ap_dbd_prepare is not called *after* post_config?
(My assumption was that it might be c
On Sunday 23 July 2006 22:35, Ruediger Pluem wrote:
> > -svr_cfg *svr = ap_get_module_config(s->module_config, &dbd_module);
> > dbd_prepared *prepared = apr_pcalloc(s->process->pool,
> > sizeof(dbd_prepared)); prepared->label = label;
> > prepared->query = query;
> > -prepared->
On 07/23/2006 10:22 PM, [EMAIL PROTECTED] wrote:
> Author: niq
> Date: Sun Jul 23 13:22:57 2006
> New Revision: 424798
>
> URL: http://svn.apache.org/viewvc?rev=424798&view=rev
> Log:
> Remove dependence on server config from ap_dbd_prepare, as that may
> be called in per-directory config functi
On 07/23/2006 07:54 PM, Ben Laurie wrote:
> Ruediger Pluem wrote:
>
>>On 07/23/2006 02:10 PM, Ben Laurie wrote:
>>
>>>Joe Orton wrote:
>>
- use APR apr_file_* not ANSI C fopen,
>>>
>>>I need a FILE *.
>>
>>Maybe you could use BIO_new_file / PEM_read_bio_PKCS7 as it is done in similar
>>situa
On 7/23/06, William A. Rowe, Jr. <[EMAIL PROTECTED]> wrote:
@@ServerRoot@@ doesn't guarantee that either :-/ grrr - please don't
start commit wars by reverting without some small post to discuss?
Here's the reply I sent to your original commit (msgid
<[EMAIL PROTECTED]>). I
didn't receive a r
@@ServerRoot@@ doesn't guarantee that either :-/ grrr - please don't
start commit wars by reverting without some small post to discuss?
[EMAIL PROTECTED] wrote:
Author: slive
Date: Sun Jul 23 14:00:58 2006
New Revision: 424817
URL: http://svn.apache.org/viewvc?rev=424817&view=rev
Log:
Backou
On 07/23/2006 09:59 PM, William A. Rowe, Jr. wrote:
> Ruediger Pluem wrote:
>
>>
>> On 07/23/2006 09:02 AM, William A. Rowe, Jr. wrote:
>>
>>> Criminy!!! If you are curious how many people really need this fix,
>>> figure
>>> only half are ready to upgrade now to 2.2.x and scroll down through t
William A. Rowe, Jr. wrote:
> Ben Laurie wrote:
>> Ruediger Pluem wrote:
>>> On 07/23/2006 02:10 PM, Ben Laurie wrote:
>
> [Sun Jul 23 10:25:14 2006] [info] Loading certificate & private key
> of SSL-aware server
> [Sun Jul 23 10:25:14 2006] [error] Can't open \x80\x94|
Hmmm. C
On 07/23/2006 08:47 PM, Nick Kew wrote:
> On Sunday 23 July 2006 16:50, [EMAIL PROTECTED] wrote:
>
>>Author: rpluem
>>Date: Sun Jul 23 08:50:08 2006
>>New Revision: 424759
>>
>>URL: http://svn.apache.org/viewvc?rev=424759&view=rev
>>Log:
>>* This shortcut is too short. It is not up to the filter
Ben Laurie wrote:
Ruediger Pluem wrote:
On 07/23/2006 02:10 PM, Ben Laurie wrote:
[Sun Jul 23 10:25:14 2006] [info] Loading certificate & private key of
SSL-aware server
[Sun Jul 23 10:25:14 2006] [error] Can't open \x80\x94|
Hmmm. Can't reproduce this (on the trunk).
This is pure luck. You
Trent Nelson wrote:
From: William A. Rowe, Jr. [mailto:[EMAIL PROTECTED]
Perhaps you are thinking of the idea of an mpm_winhttp,
using the WinHTTP API (which handles the request/response
in the WinXP HTTP stack and then dispatches between multiple
hosts and uri's that are registered with this AP
Ruediger Pluem wrote:
On 07/23/2006 09:02 AM, William A. Rowe, Jr. wrote:
Criminy!!! If you are curious how many people really need this fix, figure
only half are ready to upgrade now to 2.2.x and scroll down through the
list
of email notifications. Never seen such a monstrous list in my lif
Colm MacCarthaigh wrote:
On Sun, Jul 23, 2006 at 04:25:41AM -0500, William A. Rowe, Jr. wrote:
[EMAIL PROTECTED] wrote:
* configure.in: Add --with-included-apr flag to force use of the
bundled copies of APR and APR-util.
Any desire to have a consistent syntax? --with-expat=builtin happens
to
Ok i coudl'nt get it to work:1) the dsp file was incompatible with 2.2.x... ok no big deal I rewrote the new one by hand.2) lots of code errors! APR incompatible??... big deal since i can't debug or write c :/
The errors all seem to be related to APR functions.On 7/23/06, Mladen Turk <[EMAIL PROTEC
On Sunday 23 July 2006 19:41, Anton Golubev wrote:
> Hi Nick,
>
> I think you are right about crash reason. It probably was an
> incompatibility of http and apr-util, which I built separately to
> overcame configuration issues. Once I've had the clean build, the problem
> disappeared.
That's a rel
On Fri, 21 Jul 2006, Philip M. Gollucci wrote:
Philip M. Gollucci wrote:
Philip M. Gollucci wrote:
Randy Kobes wrote:
Which means
apr_pool_cleanup_register(pool, data,
apreq_file_cleanup, apreq_file_cleanup);
Contrary to the comment in library/util.c
data = a
On Sunday 23 July 2006 16:50, [EMAIL PROTECTED] wrote:
> Author: rpluem
> Date: Sun Jul 23 08:50:08 2006
> New Revision: 424759
>
> URL: http://svn.apache.org/viewvc?rev=424759&view=rev
> Log:
> * This shortcut is too short. It is not up to the filters to decide
> whether filters down the chain c
Hi Nick,
I think you are right about crash reason. It probably was an
incompatibility of http and apr-util, which I built separately to
overcame configuration issues. Once I've had the clean build, the problem
disappeared. There are three issues with the current
auto-configuration regarding activa
Ruediger Pluem wrote:
>
> On 07/23/2006 02:10 PM, Ben Laurie wrote:
>> Joe Orton wrote:
>
>>
>>> - use APR apr_file_* not ANSI C fopen,
>>
>> I need a FILE *.
>
> Maybe you could use BIO_new_file / PEM_read_bio_PKCS7 as it is done in similar
> situations in other places of mod_ssl.
Why?
>>> -
From: William A. Rowe, Jr. [mailto:[EMAIL PROTECTED]
> Perhaps you are thinking of the idea of an mpm_winhttp,
> using the WinHTTP API (which handles the request/response
> in the WinXP HTTP stack and then dispatches between multiple
> hosts and uri's that are registered with this API)?
Interest
Cool, I'll give it a shot... hopefully it will work on 2.2.2 aswel...If i get it to compiled... what kind of data would you be interested in?On 7/23/06,
Mladen Turk <[EMAIL PROTECTED]> wrote:
William A. Rowe, Jr. wrote:> Jorge Schrauwen wrote:>> i was talking about:>> http://mail-archives.apache.o
On 07/23/2006 02:10 PM, Ben Laurie wrote:
> Joe Orton wrote:
>
>
>>- use APR apr_file_* not ANSI C fopen,
>
>
> I need a FILE *.
Maybe you could use BIO_new_file / PEM_read_bio_PKCS7 as it is done in similar
situations in other places of mod_ssl.
>
>>- the server doesn't even start up (wi
Joe Orton wrote:
> On Sat, Jul 22, 2006 at 02:27:44PM -, [EMAIL PROTECTED] wrote:
>> Author: ben
>> Date: Sat Jul 22 07:27:43 2006
>> New Revision: 424584
>>
>> URL: http://svn.apache.org/viewvc?rev=424584&view=rev
>> Log:
>> Add PKCS#7 support.
>
> -1. We've had an RTC policy on the stable b
On 07/23/2006 09:02 AM, William A. Rowe, Jr. wrote:
> Criminy!!! If you are curious how many people really need this fix, figure
> only half are ready to upgrade now to 2.2.x and scroll down through the
> list
> of email notifications. Never seen such a monstrous list in my life after
> I clic
sön 2006-07-23 klockan 00:10 +0100 skrev Nick Kew:
> But if you look at the full traceback and crossreference it to the
> source, I think that looks improbable. Do you have sufficient gcc/gdb
> expertise to shed real light on this?
Not really, only experience..
From what I have seen the causes
I think I answered both of these on my own. For the sake of people
googling this:
After nosing around a bit more, I came across the well-named (IF you
know that it's what you're looking for!) apr_xlate.h (in
apr-util/xlate), which it seems does the job of abstracting the
underlying iconv implemen
On Sun, Jul 23, 2006 at 04:25:41AM -0500, William A. Rowe, Jr. wrote:
> [EMAIL PROTECTED] wrote:
> >
> >* configure.in: Add --with-included-apr flag to force use of the
> >bundled copies of APR and APR-util.
>
> Any desire to have a consistent syntax? --with-expat=builtin happens
> to be that dep
On Sat, Jul 22, 2006 at 02:27:44PM -, [EMAIL PROTECTED] wrote:
> Author: ben
> Date: Sat Jul 22 07:27:43 2006
> New Revision: 424584
>
> URL: http://svn.apache.org/viewvc?rev=424584&view=rev
> Log:
> Add PKCS#7 support.
-1. We've had an RTC policy on the stable branch for many years: please
[EMAIL PROTECTED] wrote:
* configure.in: Add --with-included-apr flag to force use of the
bundled copies of APR and APR-util.
Any desire to have a consistent syntax? --with-expat=builtin happens
to be that dependencies' flag to accomplish similar.
[EMAIL PROTECTED] wrote:
Author: wrowe
Date: Sun Jul 23 01:59:14 2006
New Revision: 424691
URL: http://svn.apache.org/viewvc?rev=424691&view=rev
Log:
Simplify; installtest was an -artifact-, not a design decision :-)
Well, I was not sure if you really need a separate install config
target.
William A. Rowe, Jr. wrote:
Jorge Schrauwen wrote:
i was talking about:
http://mail-archives.apache.org/mod_mbox/httpd-dev/200408.mbox/[EMAIL PROTECTED]
Just goes to show how things slip past people's radars when they have other
things going on, and really there are only two or three dev@'er
Jorge Schrauwen wrote:
i was talking about:
http://mail-archives.apache.org/mod_mbox/httpd-dev/200408.mbox/[EMAIL PROTECTED]
Just goes to show how things slip past people's radars when they have other
things going on, and really there are only two or three dev@'ers who would
have been in a pos
i was talking about:
http://mail-archives.apache.org/mod_mbox/httpd-dev/200408.mbox/[EMAIL PROTECTED]
On 7/23/06, William A. Rowe, Jr. <[EMAIL PROTECTED]> wrote:
Jorge Schrauwen wrote:
> Just wondering what happend to this?
No clue what you are asking about. Perhaps you are thinking of the
ide
[EMAIL PROTECTED] wrote:
Log:
I really hated that bug's title :)
-* mod_ssl: Solve SSLVerifyClient in
- (This was already committed to 2.2.x)
+* mod_ssl: Solve POST incompatible w/ renegotiate HTTPS connection
+ (This was already committed to 2.2.x, and the reports persist.
51 matches
Mail list logo