CORS Enable Headers about tus-node-server HOT 8 CLOSED

Since there was no Location info, the URL for PATCH was null. That was the actual error.
Now everything seems to be OK, thank you!

from tus-node-server.

Ah yes. A fix could be similar to 'Access-Control-Allow-Headers' in the OptionsHandler.js.

I can submit a patch, unless you have a pull request ready to go?

Thanks @feluna !

from tus-node-server.

You can submit a patch, thank you as well.

from tus-node-server.

@feluna if you update to [email protected] you should be good to go 👍

from tus-node-server.

Hi, I'm using 0.1.1, but still had to include the extra line from the original post to make the missing header part appear and the file uploads resume.

from tus-node-server.

@krampampuli I realized a case where it wouldn't be added to the HEAD request, which is fixed in [email protected]. Let me know if you have any trouble after upgrading!

from tus-node-server.

After upgrade it gives "405 Method Not Allowed: The requested method PATCH is not allowed for the URL /null."

Exposed headers seem to miss Location, i.e., it should be added to const HEADERS in constants.js

Thanks!

from tus-node-server.

@krampampuli thanks for your help. Location is added in [email protected].

Not sure I'm following how PATCH isn't allowed, since it is included in the Access-Control-Allow-Methods of OPTIONS request. Would you mind opening up a separate ticket with detailed steps to replicate?

from tus-node-server.