From 73bff1ba191cd3b31546232397d2a6c535b880c2 Mon Sep 17 00:00:00 2001 From: tsodring Date: Mon, 24 Jul 2023 15:22:46 +0200 Subject: [PATCH] Remove Allows header check It is not clear that we should be checking for the ALLOW header when testing. This was discussed in [27](https://github.com/petterreinholdtsen/noark5-tester/issues/27). Allow is part of CORS and we are likely using it wrongly. The proposed way of dealing with this is to remove it until we know what we want to achieve with it. --- runtest | 2 -- 1 file changed, 2 deletions(-) diff --git a/runtest b/runtest index 0164e7a..c5e81fa 100755 --- a/runtest +++ b/runtest @@ -375,9 +375,7 @@ class Noark5Tester (n5core.endpoint.Endpoint): urlseen[url] = 1 try: (content, res) = self.json_get(url) - allow = res.getheader('Allow') self.success("GET %s worked with code %d" % (url, res.code)) - self.verify(allow is not None, "GET header should include Allow for %s" % url) ctype = res.getheader('Content-Type') if not ctype: