Am Sunday 11 November 2012 schrieb Giuseppe Scrivano:
> Tim Ruehsen <[email protected]> writes:
> > To check wget for NOT sending a request header, I patched HTTPServer.pm.
> > Here is an example taken from the patched Test-cookies.px (see below):
> > ...
> > 
> >         request_headers => {
> >         
> >             "!Cookie" => qr|foo=bar|,
> >         
> >         },
> > 
> > ...
> > 
> > That means: return an error, if there is a Cookie request header that
> > contains foo=bar. (I hope, I understood that perl stuff correctly).
> > I am not perl programmer, so maybe my changes are suboptimal. Please
> > someone check that.
> > 
> > 
> > Then, I expanded Test-cookies.px to delete the session cookie by using an
> > outdated 'Expires' tag. (works fine).
> > 
> > Also included is a test for testing the ignorance of a non-matching
> > domain in the Set-Cookie header. (wget fails, but shouldn't due to RFC
> > 6265 5.3.6). wget just replaces that foreign domain with the domain from
> > the request-uri.
> > 
> > 
> > Should things like that go to the bug tracker ? (There are so many open
> > issues for so long time that I am unshure about that.)
> 
> I think few people are following the activity on Savannah, I don't see
> problems to send these generic fixes here on the mailing list and give
> them a possibility to comment.
> 
> The patches seem ok, thanks for that.  Could you please complete them
> with the ChangeLog entries?

Sorry, here it comes.

Tim
From fe9609eb581e4f5c40e05a12dda90b816595a849 Mon Sep 17 00:00:00 2001
From: Tim Ruehsen <[email protected]>
Date: Mon, 12 Nov 2012 09:04:41 +0100
Subject: [PATCH] update tests/ChangeLog

---
 tests/ChangeLog |    5 +++++
 1 file changed, 5 insertions(+)

diff --git a/tests/ChangeLog b/tests/ChangeLog
index e12d8b5..3650f82 100644
--- a/tests/ChangeLog
+++ b/tests/ChangeLog
@@ -1,3 +1,8 @@
+2012-11-09  Tim Ruehsen  <[email protected]>
+
+	* HTTPServer.pm: added check for must-not-match request-header
+	* Test-cookies.px: check cookie deletion and cookie domain matching
+
 2012-06-16  Giuseppe Scrivano  <[email protected]>
 
 	* Makefile.am (EXTRA_DIST): Add Test-stdouterr.px.
-- 
1.7.10.4

Reply via email to