Incorrect file name mangling with dual extensions files about pocketbase HOT 3 CLOSED

I'll investigate it more details tomorrow and eventually will consider preserving double extensions.

from pocketbase.

I've pushed a fix in master to take double extensions in consideration and it will be available with the next minor v0.22.10 release (most likely tomorrow or the day following it).

Regarding the hook, __hooks is a string and doesn't have StopPropagation field. To stop the hook chain in the JSVM you need to either throw an error OR return false (note that the hooks will slightly change with the refactoring).

from pocketbase.

@ganigeorgiev thank you for your quick response. In the meantime, I've found the code snippet for changing the filename on the fly using hook in #306 (comment)

I'm trying to translate it to Javascript without success.

This is what I had so far:

onFileDownloadRequest((e) => {
  fs = $app.newFilesystem();
  fs.serve(e.httpContext.response(), e.servedPath, "test.zip");
  fs.close();

  return __hooks.StopPropagation;
});

Any tips?

from pocketbase.