cgit 1.2.3: lighttpd 1.4.57, AlpineLinux [edge]: using cache breaks delivery

[Steffen Nurpmeso]

Hello.

Steffen Nurpmeso wrote in
 <20201221193127.zbZeP%steffen at sdaoden.eu>:
 |John Keeping wrote in
 | <X+DiDgGaPaynnocI at john.keeping.me.uk>:
 ||On Mon, Dec 21, 2020 at 05:26:19PM +0100, Steffen Nurpmeso wrote:
 ||> I discovered today that cgit no longer delivers pages, and it must
 ||> have been like that for some time.  The server looks show
 ||> successful delivery, the cgit cache is populated and rotated just
 ||> correctly, but all cgit delivers is that final error of main() as
 ||> 
 ||>   <div class='error'>Error processing page: Invalid argument (22)</div>
 ...
 ||> I am pretty sure cgit delivered some weeks ago, the most notable
 ||> difference is that AlpineLinux switched to Lighttpd 1.4.56 then
 ||> .57, which seems to have brought tremendous changes under the
 ...
 |But the file was generated normally:
 |
 |  # ll /var/lib/lighttpd/cgit/b1000000
 |  -rw-------    1 lighttpd lighttpd     23417 Dec 21 20:22 /var/lib/lightt\
 |  pd/cgit/b1000000
 ...

Slightly resorted:

 ...
 ||Have you looked at the log output?
 |  [cgit] error printing cache /var/lib/lighttpd/cgit/b1000000: Invalid \
 |  argument (22)
 ...
 ||and this may be caused by sendfile(2) failing due to some difference in
 ||how the web server is setting up the output file descriptor.  You may
 ||want to rebuild CGit without HAVE_LINUX_SENDFILE and see if that works.

So i build it with

  #alp-2020:$ diff cgit.mk.orig cgit.mk
  --- cgit.mk.orig
  +++ cgit.mk
  @@ -64,7 +64,7 @@
   endif

   ifdef HAVE_LINUX_SENDFILE
  -       CGIT_CFLAGS += -DHAVE_LINUX_SENDFILE
  +       #CGIT_CFLAGS += -DHAVE_LINUX_SENDFILE
   endif

   CGIT_OBJ_NAMES += cgit.o

and

  make NO_LUA=y NO_ICONV=y NO_GETTEXT=y NO_TCLTK=y NO_PERL=1 \
    NO_PYTHON=1 NO_SVN_TESTS=y NO_REGEX=NeedsStartEnd prefix=/usr

and .. it works.

Thank you, i will open an AlpineLinux bug report.  And lighttpd.

--steffen
|
|Der Kragenbaer,                The moon bear,
|der holt sich munter           he cheerfully and one by one
|einen nach dem anderen runter  wa.ks himself off
|(By Robert Gernhardt)