Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: server rewrite code #2

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Conversation

xiaobiaozhao
Copy link
Owner

Why do we need this PR?

Currently, only NGX_HTTP_REWRITE_PHASE has lua modules. If you need use lua code in NGX_HTTP_SERVER_REWRITE_PHASE, you need to develop nginx C modules. So I make this PR solve the problem

Add 3 directive

  1. server_rewrite_by_lua
  2. server_rewrite_by_lua_block
  3. server_rewrite_by_lua_file

I hereby granted the copyright of the changes in this pull request
to the authors of this lua-nginx-module project.

@xiaobiaozhao xiaobiaozhao force-pushed the feature_server_rewrite branch 5 times, most recently from 97f4547 to d84ed91 Compare February 25, 2022 16:40
@zhuizhuhaomeng zhuizhuhaomeng force-pushed the feature_server_rewrite branch from 3f050db to d7ac596 Compare March 7, 2022 12:11
spacewander and others added 13 commits March 8, 2022 08:40
Signed-off-by: spacewander <spacewanderlzx@gmail.com>
…k:sslhandshake` with FFI

This adds support for setting client certificate/private key that will be used later
for mutual TLS handshake with a server. Also, the `tcpsock:sslhandshake`
implementation has been rewritten to use FFI C API to be more performant
and easier to maintain.

Co-authored-by: Chrono Law <chrono.law@konghq.com>
add 2 directive
1. server_rewrite_by_lua_block
2. server_rewrite_by_lua_file
@zhuizhuhaomeng zhuizhuhaomeng force-pushed the feature_server_rewrite branch from ac8ed38 to 900a3c2 Compare March 19, 2022 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants