Return-Path: Delivered-To: apmail-httpd-dev-archive@www.apache.org Received: (qmail 12087 invoked from network); 19 Oct 2005 20:00:13 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 19 Oct 2005 20:00:13 -0000 Received: (qmail 29861 invoked by uid 500); 19 Oct 2005 20:00:03 -0000 Delivered-To: apmail-httpd-dev-archive@httpd.apache.org Received: (qmail 29778 invoked by uid 500); 19 Oct 2005 20:00:03 -0000 Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm Precedence: bulk Reply-To: dev@httpd.apache.org list-help: list-unsubscribe: List-Post: List-Id: Delivered-To: mailing list dev@httpd.apache.org Received: (qmail 29754 invoked by uid 99); 19 Oct 2005 20:00:03 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 19 Oct 2005 13:00:03 -0700 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received-SPF: neutral (asf.osuosl.org: local policy) Received: from [196.31.24.162] (HELO gatekeeper.fma.co.za) (196.31.24.162) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 19 Oct 2005 13:00:02 -0700 Received: from localhost (localhost.localdomain [127.0.0.1]) by gatekeeper.fma.co.za (Postfix) with ESMTP id AD17CB20BD for ; Wed, 19 Oct 2005 21:59:29 +0200 (SAST) Received: from gatekeeper.fma.co.za ([127.0.0.1]) by localhost (gatekeeper.fma.co.za [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 15525-01 for ; Wed, 19 Oct 2005 21:59:27 +0200 (SAST) Received: from [196.30.143.210] (gatekeeper.fma.co.za [196.30.143.210]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by gatekeeper.fma.co.za (Postfix) with ESMTP id CCFA6B1C60 for ; Wed, 19 Oct 2005 21:59:25 +0200 (SAST) Message-ID: <4356A59A.4060303@sharp.fm> Date: Wed, 19 Oct 2005 21:59:22 +0200 From: Graham Leggett User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.12) Gecko/20050921 Red Hat/1.7.12-1.4.1 X-Accept-Language: en-za, en-us, en MIME-Version: 1.0 To: dev@httpd.apache.org Subject: Re: [PATCH] PR 18757 Content-Length removed from proxied HEAD responses References: <43557DFF.9040006@apache.org> <435667B3.5020904@apache.org> <7941cfb11f55f11347d256bd178a3fa3@gbiv.com> In-Reply-To: <7941cfb11f55f11347d256bd178a3fa3@gbiv.com> Content-Type: multipart/signed; protocol="application/x-pkcs7-signature"; micalg=sha1; boundary="------------ms020802080706000104050605" X-Virus-Scanned: by amavisd-new at fma.co.za X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N This is a cryptographically signed message in MIME format. --------------ms020802080706000104050605 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Roy T. Fielding wrote: > That sounds like spaghetti code to me. Fix the bug in the filters, > not the side-effects. > > +1 on this patch as a temporary fix if you add commentary with it, > but I'd rather have a server that isn't so fragile in the long term. To my knowledge filters have never had the concept of metadata that goes alongside the bucket brigades, for example the content-length, the mime type, any range(s) associated with the request. It would be useful (and generic) for filters to have a way of signalling reliably whether the filter changes the length of a request, or the mime type of the payload, or whatever, without being HTTP specific. Proxy has no business caring what an upstream filter might do to a request, it should just load the metadata with it's best knowledge of what's on it's way up the stack, and rely on filters (if any) to modify the metadata along the way. Metadata should also be "lockable", in other words once the HTTP headers are written out to the network, the metadata is "locked" in place and cannot be changed, attempts at doing so will trigger an error in the request. This should catch attempts by filters to change metadata like content length after it's too late. Regards, Graham -- --------------ms020802080706000104050605 Content-Type: application/x-pkcs7-signature; name="smime.p7s" Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="smime.p7s" Content-Description: S/MIME Cryptographic Signature MIAGCSqGSIb3DQEHAqCAMIACAQExCzAJBgUrDgMCGgUAMIAGCSqGSIb3DQEHAQAAoIIJGzCC AugwggJRoAMCAQICAw+EzzANBgkqhkiG9w0BAQQFADBiMQswCQYDVQQGEwJaQTElMCMGA1UE ChMcVGhhd3RlIENvbnN1bHRpbmcgKFB0eSkgTHRkLjEsMCoGA1UEAxMjVGhhd3RlIFBlcnNv bmFsIEZyZWVtYWlsIElzc3VpbmcgQ0EwHhcNMDUwOTIxMDkzODE2WhcNMDYwOTIxMDkzODE2 WjBdMRAwDgYDVQQEEwdMZWdnZXR0MQ8wDQYDVQQqEwZHcmFoYW0xFzAVBgNVBAMTDkdyYWhh bSBMZWdnZXR0MR8wHQYJKoZIhvcNAQkBFhBtaW5mcmluQHNoYXJwLmZtMIIBIjANBgkqhkiG 9w0BAQEFAAOCAQ8AMIIBCgKCAQEAxDGvVYuOXeBGu0PSArbW5SSSzztjMg1DPP90vksG/Y8Y gLf9MZ+NWKMf8BvWa30HOJJU9slrqbgEcIJPNpdtuJ3bG7+SIhPpibaw1pYhPWr06kbSaRH2 W0FBLZC5/pwseUnqbfLSqkfjd3NBe4k/3qaRr534gnmrwOdlUzwYHPZfXfLslvJM+14EFSDN s/AEphlW5704v+Ucb0Xe5IkJQzyy8DhU19BBqjqGdhykVv5wgJLiK/85BdYi/dfLwQOzzsbd 1HB/CJcSphBfmc24YPM+Sj5w8cX1gUnbTKhvHvQM4hs8gazsqb0A1lVnVLLeL0dlWnAEu/dt YDoeeDU8HwIDAQABoy0wKzAbBgNVHREEFDASgRBtaW5mcmluQHNoYXJwLmZtMAwGA1UdEwEB /wQCMAAwDQYJKoZIhvcNAQEEBQADgYEAlVOZib28O/HqUOp/ZMAxsJ5Ecm2EJI9v26hoITkx t+YrnwH11d59iNMiTvvGpPiZEtJiNdxqyNol/Jh25SZSilY9iGfCdOB3A5aLvQOCOWVhsiut Ly0gI4+4fOY5YNjCDK9K8by9vVTjhEHfnRaUlsLaUfE1xE2Ed9GIUsj72h8wggLoMIICUaAD AgECAgMPhM8wDQYJKoZIhvcNAQEEBQAwYjELMAkGA1UEBhMCWkExJTAjBgNVBAoTHFRoYXd0 ZSBDb25zdWx0aW5nIChQdHkpIEx0ZC4xLDAqBgNVBAMTI1RoYXd0ZSBQZXJzb25hbCBGcmVl bWFpbCBJc3N1aW5nIENBMB4XDTA1MDkyMTA5MzgxNloXDTA2MDkyMTA5MzgxNlowXTEQMA4G A1UEBBMHTGVnZ2V0dDEPMA0GA1UEKhMGR3JhaGFtMRcwFQYDVQQDEw5HcmFoYW0gTGVnZ2V0 dDEfMB0GCSqGSIb3DQEJARYQbWluZnJpbkBzaGFycC5mbTCCASIwDQYJKoZIhvcNAQEBBQAD ggEPADCCAQoCggEBAMQxr1WLjl3gRrtD0gK21uUkks87YzINQzz/dL5LBv2PGIC3/TGfjVij H/Ab1mt9BziSVPbJa6m4BHCCTzaXbbid2xu/kiIT6Ym2sNaWIT1q9OpG0mkR9ltBQS2Quf6c LHlJ6m3y0qpH43dzQXuJP96mka+d+IJ5q8DnZVM8GBz2X13y7JbyTPteBBUgzbPwBKYZVue9 OL/lHG9F3uSJCUM8svA4VNfQQao6hnYcpFb+cICS4iv/OQXWIv3Xy8EDs87G3dRwfwiXEqYQ X5nNuGDzPko+cPHF9YFJ20yobx70DOIbPIGs7Km9ANZVZ1Sy3i9HZVpwBLv3bWA6Hng1PB8C AwEAAaMtMCswGwYDVR0RBBQwEoEQbWluZnJpbkBzaGFycC5mbTAMBgNVHRMBAf8EAjAAMA0G CSqGSIb3DQEBBAUAA4GBAJVTmYm9vDvx6lDqf2TAMbCeRHJthCSPb9uoaCE5MbfmK58B9dXe fYjTIk77xqT4mRLSYjXcasjaJfyYduUmUopWPYhnwnTgdwOWi70DgjllYbIrrS8tICOPuHzm OWDYwgyvSvG8vb1U44RB350WlJbC2lHxNcRNhHfRiFLI+9ofMIIDPzCCAqigAwIBAgIBDTAN BgkqhkiG9w0BAQUFADCB0TELMAkGA1UEBhMCWkExFTATBgNVBAgTDFdlc3Rlcm4gQ2FwZTES MBAGA1UEBxMJQ2FwZSBUb3duMRowGAYDVQQKExFUaGF3dGUgQ29uc3VsdGluZzEoMCYGA1UE CxMfQ2VydGlmaWNhdGlvbiBTZXJ2aWNlcyBEaXZpc2lvbjEkMCIGA1UEAxMbVGhhd3RlIFBl cnNvbmFsIEZyZWVtYWlsIENBMSswKQYJKoZIhvcNAQkBFhxwZXJzb25hbC1mcmVlbWFpbEB0 aGF3dGUuY29tMB4XDTAzMDcxNzAwMDAwMFoXDTEzMDcxNjIzNTk1OVowYjELMAkGA1UEBhMC WkExJTAjBgNVBAoTHFRoYXd0ZSBDb25zdWx0aW5nIChQdHkpIEx0ZC4xLDAqBgNVBAMTI1Ro YXd0ZSBQZXJzb25hbCBGcmVlbWFpbCBJc3N1aW5nIENBMIGfMA0GCSqGSIb3DQEBAQUAA4GN ADCBiQKBgQDEpjxVc1X7TrnKmVoeaMB1BHCd3+n/ox7svc31W/Iadr1/DDph8r9RzgHU5VAK MNcCY1osiRVwjt3J8CuFWqo/cVbLrzwLB+fxH5E2JCoTzyvV84J3PQO+K/67GD4Hv0CAAmTX p6a7n2XRxSpUhQ9IBH+nttE8YQRAHmQZcmC3+wIDAQABo4GUMIGRMBIGA1UdEwEB/wQIMAYB Af8CAQAwQwYDVR0fBDwwOjA4oDagNIYyaHR0cDovL2NybC50aGF3dGUuY29tL1RoYXd0ZVBl cnNvbmFsRnJlZW1haWxDQS5jcmwwCwYDVR0PBAQDAgEGMCkGA1UdEQQiMCCkHjAcMRowGAYD VQQDExFQcml2YXRlTGFiZWwyLTEzODANBgkqhkiG9w0BAQUFAAOBgQBIjNFQg+oLLswNo2as Zw9/r6y+whehQ5aUnX9MIbj4Nh+qLZ82L8D0HFAgk3A8/a3hYWLD2ToZfoSxmRsAxRoLgnSe JVCUYsfbJ3FXJY3dqZw5jowgT2Vfldr394fWxghOrvbqNOUQGls1TXfjViF4gtwhGTXeJLHT HUb/XV9lTzGCAzswggM3AgEBMGkwYjELMAkGA1UEBhMCWkExJTAjBgNVBAoTHFRoYXd0ZSBD b25zdWx0aW5nIChQdHkpIEx0ZC4xLDAqBgNVBAMTI1RoYXd0ZSBQZXJzb25hbCBGcmVlbWFp bCBJc3N1aW5nIENBAgMPhM8wCQYFKw4DAhoFAKCCAacwGAYJKoZIhvcNAQkDMQsGCSqGSIb3 DQEHATAcBgkqhkiG9w0BCQUxDxcNMDUxMDE5MTk1OTIyWjAjBgkqhkiG9w0BCQQxFgQU45nB 1249U32u2E8F8e+stsuCSl0wUgYJKoZIhvcNAQkPMUUwQzAKBggqhkiG9w0DBzAOBggqhkiG 9w0DAgICAIAwDQYIKoZIhvcNAwICAUAwBwYFKw4DAgcwDQYIKoZIhvcNAwICASgweAYJKwYB BAGCNxAEMWswaTBiMQswCQYDVQQGEwJaQTElMCMGA1UEChMcVGhhd3RlIENvbnN1bHRpbmcg KFB0eSkgTHRkLjEsMCoGA1UEAxMjVGhhd3RlIFBlcnNvbmFsIEZyZWVtYWlsIElzc3Vpbmcg Q0ECAw+EzzB6BgsqhkiG9w0BCRACCzFroGkwYjELMAkGA1UEBhMCWkExJTAjBgNVBAoTHFRo YXd0ZSBDb25zdWx0aW5nIChQdHkpIEx0ZC4xLDAqBgNVBAMTI1RoYXd0ZSBQZXJzb25hbCBG cmVlbWFpbCBJc3N1aW5nIENBAgMPhM8wDQYJKoZIhvcNAQEBBQAEggEATCmzK2q6Fk891GM6 ZY8C+ZvNACEDvateEAXSVDuQqtILoNOLMN0JWxm0gycF0CybW9Y9MQsSwn+Au+Oogd9IuqNO XvdSs8C69SuQQndVWXy4tDtOiLMsffhfQ/f+fR1k7zAra+CNt7R01xv+uGODIAjxDIF891eZ No1cD5jz2AdDqGw1o5gJ3xOjtj44RECN1v0A0wOTgJbI9wfvoZRXXjETP8KyaV3RX69h8Gtj 0bWXVKiru39tzm88p1SUeDnxvzib4nfL46rvVPe9ZVaBaQjMYvVnGvH6jFKKTf+LEdcp9xNU cYG6hXIhVkoqm8N5tGXynDEn7FyVONtBN/URcgAAAAAAAA== --------------ms020802080706000104050605--