Support OPTIONS Request and CORS Headers about prosody-filer HOT 9 CLOSED

CORS now available via 22c2c4f (Release v1.0.0)

from prosody-filer.

CORS headers not recognized by https://compliance.conversations.im/ because of https://github.com/iNPUTmice/caas/issues/43

from prosody-filer.

CORS works, proven by https://compliance.conversations.im/server/anoxinon.me/

from prosody-filer.

CORS headers work on https://compliance.conversations.im/ ✔️

from prosody-filer.

Is there anything that needs to be enable in nginx ?

I am asking because here is what I get from conversesj:

Access to XMLHttpRequest at 'https://myserver.com/upload/665b1e62-b8b17f-94c5fe25bd38/file.pdf?v=afa2e83efbb49cb5ca311c2a11c91d3338b3e6e34e32a528552' from origin 'https://www.myserver.com' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: It does not have HTTP ok status.

from prosody-filer.

@meaz I'm having the same issue with my own Converse.js instance at the moment. Maybe I have some free time on the upcoming days to investigate this issue and suggest a solution. Either in the Nginx side or the Prosody Filer side.

from prosody-filer.

OPTIONS request is now supported via new Nginx config:

[...]
location /upload/ {
        if ( $request_method = OPTIONS ) {
                add_header Access-Control-Allow-Origin '*';
                add_header Access-Control-Allow-Methods 'PUT, GET, OPTIONS, HEAD';
                add_header Access-Control-Allow-Headers 'Authorization, Content-Type';
                add_header Access-Control-Allow-Credentials 'true';
                add_header Content-Length 0;
                add_header Content-Type text/plain;
                return 200;
        }

        proxy_pass http://127.0.0.1:5050/upload/;
        proxy_request_buffering off;
    }
[...]

(also see README.md in master branch)

from prosody-filer.

Thanks a lot @ThomasLeister

There is small error on the Readme:

location /upload/ {
        location /upload/ {

should just be:
location /upload/ {

from prosody-filer.

Thank you, I just fixed it :-)

from prosody-filer.