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

bugfix: ensured arguments of APIs mutating uri or request/response headers do not contain unsafe characters. #1654

Merged
merged 1 commit into from Mar 19, 2020

Conversation

doujiang24
Copy link
Member

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

src/ngx_http_lua_uri.c Outdated Show resolved Hide resolved
src/ngx_http_lua_headers_in.c Outdated Show resolved Hide resolved
src/ngx_http_lua_uri.c Outdated Show resolved Hide resolved
src/ngx_http_lua_uri.c Outdated Show resolved Hide resolved
src/ngx_http_lua_util.h Outdated Show resolved Hide resolved
@thibaultcha thibaultcha force-pushed the set_uri branch 2 times, most recently from b13227f to 215647d Compare March 9, 2020 20:12
@thibaultcha
Copy link
Member

thibaultcha commented Mar 9, 2020

(Rebased on top of master due to a newly failing tests caused by changes in openresty.org's accepted ciphers)

@thibaultcha
Copy link
Member

Just pushed an additional commit with more fixes.

…aders do not contain unsafe characters.

Signed-off-by: Thibault Charbonnier <thibaultcha@me.com>
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.

None yet

3 participants