On 09/06/22 14:00, Harald Oehlmann wrote:

Am 09.06.2022 um 12:37 schrieb Massimo Manghi:
Rivet 3.2.2 rc1 is available from

http://www.rivetweb.org/~manghi/rivet/rivet-3.2.2rc1.tar.gz

New features and bug fixes

  + a memory leak was fixed in Rivet_Var (src/mod_rivet_ng/rivetCore.c)

  + Installation and TEA scripts were updated

  + The form [::rivet::var all] accepts an optional flat list argument of key-value pairs which are defaults for an application query variables (yes, this argument must be interpreted as a dictionary). This form generalizes the [::rivet::var get <varname> <default>] form.

  + Tests for this new form were added and also ::rivet::var tests in general were improved

  + Configuration directive HonorHeaderOnlyReqs, though still supported, was renamed as HonorHeadRequests. Speaking the HTTP related parlance I think this a better wording of this directive (suggestions? Like dropping support for this directive and honor HTTP head requests every time?)

Dear Massimo,

thank you for the great and detail work.
I am in favor of all those changes.

My thought about the head request: head requests are standard for http since the beginning. I would just honor them always. They are part of http and should be supported in a standard way.

Thank you,
Harald



The switch has been around ever since and it's only function is to force mod_rivet to return a "text/html" message avoiding actual script execution.

    if (private->r->header_only &&
       !private->running_conf->honor_head_requests)
    {
        TclWeb_SetHeaderType(DEFAULT_HEADER_TYPE, private->req);
        TclWeb_PrintHeaders(private->req);
        retval = OK;
        goto sendcleanup;
    }


Thus no meaningful information is returned if HonorHeadRequests is false (default)

 -- Massimo

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to