Some (but not all) static files fail to be served - ERR_CONTENT_LENGTH_MISMATCH about opine

[xyzshantaram]

@cmorten -- Here you go! https://github.com/xyzshantaram/opine-content-length-bug

Sorry I couldn't work on it last year ;)

[xyzshantaram]

@jeremythille Hi! I've run into the same issue, and I'm wondering if you've found a workaround yet -- this is kind of a deal breaker...

[jeremythille]

@ No, actually I tried to remove Opine altogether and serve my files just with Deno, without a library, and so far it works well

[cmorten]

Heya @xyzshantaram, @jeremythille 👋

Do you have a minimal reproduceable example / gist / repo that I can use to try and debug this issue?

[jeremythille]

Ha, no sorry (and it's new year's eve, and then I'm going on a trip so I don't really have time to set up a repo now). But I've put my whole server code in my original question, apart from the Angular files which are really generic HTML/CSS/JS files. Thanks for looking into this issue, and of course, happy new year :)

[xyzshantaram]

@jeremythille That will work for a bit, I suppose.
@cmorten I can try to write one for you, it's a bit late where I am but I will try to post it by tomorrow!

[cmorten]

Thanks for the great repro @xyzshantaram!

I've identified the issue - with some of the changes moving to the new std/http server for 2.0.0 of opine a regression has been introduced where served files are now being closed before the stream has finished reading them.

I have an idea for a fix which can look into this afternoon 😄

The code has its own reader impl. which was needed originally, and to ensure resource cleanup this close loop was introduced. With the 2.0.0 changes we now use a readable stream for the response which is capable of closing the resource itself- suspect can bin off the hand rolled reader to fix the bug and also get a slight perf benefit 😄

[cmorten]

Hey @jeremythille @xyzshantaram I've just released 2.0.2 which I believe fixes the issue. If not please do reopen!

[jeremythille]

Hi Craig, thanks for the follow-up and the quick maintenance. Will try to test it later today or tomorrow. Cheers

…

On Sat, 1 Jan 2022 at 19:48, Craig Morten ***@***.***> wrote: Hey @jeremythille <https://github.com/jeremythille> @xyzshantaram <https://github.com/xyzshantaram> I've just released 2.0.2 which I believe fixes the issue. If not please do reopen! — Reply to this email directly, view it on GitHub <#151 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AA4JAIEL7HBPS7XWMI6LYGLUT5EBXANCNFSM5KI62V4A> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you were mentioned.Message ID: ***@***.***>

[xyzshantaram]

Ah, crap, I forgot to respond to this! @cmorten thanks a lot for the prompt fix, it worked perfectly! <3

[jeremythille]

For the record, I still haven't tried @cmorten's fix, because I'm still trying to code a little server using native Deno code (for learning purposes, I want to understand how Deno works before using libraries), and I have the same issue without Opine.

For that matter, I have just opened a similar issue on Deno's repo: denoland/deno#13362

So... is it an Opine, or a Deno issue after all? :)

EDIT

It is a "me" issue! I forgot to encapsulate the inner await in an anonymous function, it works now. Sorry folks.