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

Sign plus in URL after '?' not converted to space #98

Closed
rekzi opened this issue Aug 29, 2017 · 4 comments
Closed

Sign plus in URL after '?' not converted to space #98

rekzi opened this issue Aug 29, 2017 · 4 comments
Assignees
Labels
bug Something isn't working question Further information is requested
Milestone

Comments

@rekzi
Copy link

rekzi commented Aug 29, 2017

If you are russian spoken, look - MailRuChamps/hlcupdocs#81

@dedok
Copy link
Contributor

dedok commented Aug 29, 2017

Hi!
Can you send me a test case?

@dedok dedok self-assigned this Aug 29, 2017
@dedok dedok added question Further information is requested bug Something isn't working labels Aug 29, 2017
@dedok dedok added this to the v2.6 milestone Aug 29, 2017
@rekzi
Copy link
Author

rekzi commented Aug 30, 2017

*.conf

upstream backend {
	server 127.0.0.1:1111;
}
server { #...
location /get_something {
      tnt_http_rest_methods post;
      tnt_pass_http_request on parse_args unescape pass_body;
      tnt_method get_something;
      tnt_pass backend;
      tnt_pure_result on;
      internal;
}
location /open {
      lua_need_request_body on;
      rewrite_by_lua_block {
       
local res = ngx.location.capture('/get_something', {
    args = ngx.var.args,
    method = ngx.HTTP_POST,
    always_forward_body = true
})

-- ...
      }
}}

Tarantool Lua script

function get_something (req)
    local get_arg = req.args.getArg
    -- ?getArg=a+b => string 'a+b', but must be 'a b'
end

@dedok
Copy link
Contributor

dedok commented Sep 1, 2017

Thx you! I will fix this issue after my vacation end, that will happen in next monday.

@dedok
Copy link
Contributor

dedok commented Dec 10, 2017

Fixed in b108c1e

@dedok dedok closed this as completed Dec 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants