rewrite handler does not handle write event correctly with thread pool mode or coroutine mode about nginx-clojure

[ekubacki]

I am seeing an issue that is similar but it's also happening in async mode as well. I am getting the following error from nginx:
Invalid URL prefix in "should_get_overwritten", client: 127.0.0.1, server: localhost, request: "GET / HTTP/1.1", host: "localhost:8080"

It's as if the rewrite handler is never getting called which results in the variable never being rewritten. Please see below for a gist of my setup.

here is a gist from my nginx.conf

http {
... 
  server {
        ....
        set $destination "should_get_overwritten";
        location / {
           handler_type 'java';
           rewrite_handler_name 'my.handlers.ProxyHandler';
           proxy_pass $destination;
        }
       ...
  }
....
}

ProxyHandler gist:

public class ProxyHandler implements NginxJavaRingHandler {
    private static final String DESTINATION_HOST = "destination";
    private static final String URL = "http://google.com";

    @Override
    public Object[] invoke(Map<String, Object> request) {
        System.err.println("============= Inside invoke =============");
        String destinationHost = determineDestinationHost();
        NginxJavaRequest r = ((NginxJavaRequest)request);

        NginxClojureRT.setNGXVariable(r.nativeRequest(), DESTINATION_HOST, destinationHost);

        return PHRASE_DONE;
    }

    private String determineDestinationHost() { return URL; }
}

few notes from documentation inaccuracies:

1. documentation calls it: handler_rewrite_name which i have sense fixed to be called rewrite_handler_name
2. I have to cast the incoming request as a NginxJavaRequest to get the nativeRequest in the java handler. This is not documented in the example provided

Intentions of what I am trying to do:
I am trying to achieve setting up a dynamically programmable reverse proxy

[xfeep]

@ekubacki thank you very much!
I'll correct the document and try your example soon.

[xfeep]

@ekubacki this issue won't happend in default mode.
You example meets issue #45 , I have fixed it just now by commit 6afeda3. The document has been corrected too.
Thanks again!

[ekubacki]

Thank-you for the correction in documentation :)

[xfeep]

close it and we release v0.2.6 now.