Include LUA for nginx when building ESP

[aashish004]

Is it possible to include LUA for nginx when building ESP

[qiwzhang]

@PiotrSikora your take on this?

[PiotrSikora]

It's possible, since Lua is just another module, that can be built separately from the core, but it requires creating and maintaining BUILD files for it.

[aashish004]

@PiotrSikora - Do you think it is worth building and maintaining this?

[PiotrSikora]

@aashish004 that depends on your use case, really.

[aashish004]

My use case is validation of JWT via cookie, queryParams, or Header (Last one is supported by ESP). The motivation here is to do authentication for all types of clients in one place so that they don't have to do it also for web clients cookie is much secure for me.

[aashish004]

Also If you can please point me to some documentation on how to include external modules while building ESP.

[qiwzhang]

If just for jwt validation, you can file enhancement issues specifically. ESP currently supports authorization header and "access_token=" query parameter. It doesn't support cookie now, but should be easy to add. The code is here: https://github.com/cloudendpoints/esp/blob/master/src/api_manager/check_auth.cc#L165

[qiwzhang]

ESP gets its Nignx with Bazel supports from here. I am not sure if it supports external contributions.

Another way is to add the Lua module in ESP. The task is: to add bazel dependencies, bazel BUILD file to build it, and add it to the src/nginx/main/module.cc. You can try to play with it, if it works, submit a PR.