Delete multiple dynamically named cookies

asaxena001 · 2016-01-12T19:27:54+00:00

Hello!I've noticed that when calling ngx.exit() within a header_filter block, the filter seems to run multiple times. Given the following config: &...

Delete multiple dynamically named cookies

asaxena001

Below code works fine:

ngx.header["Set-Cookie"] = {
   'test1=; expires=Thu, Jan 01 1970 00:00:00 UTC; domain=test.com;',
   'test2=; expires=Thu, Jan 01 1970 00:00:00 UTC; domain=test.com;'
}

though when trying to make the cookie names dynamic, it doesn't work:

local cookies = {}
local args = {'test1', 'test2'}

for i=1, #args do
  cookies[i] = args[i] .. '=; expires=Thu, Jan 01 1970 00:00:00 UTC; domain=test.com;'
end

ngx.header["Set-Cookie"] = cookies

have even tried using table.insert:

local cookies = {}
local args = {'test1', 'test2'}

for i=1, #args do
  table.insert(cookies, args[i] .. '=; expires=Thu, Jan 01 1970 00:00:00 UTC; domain=test.com;')
end

ngx.header["Set-Cookie"] = cookies

Issue seems with the assignment of variable to ngx.header["Set-Cookie"]

asaxena001

Using ngx.header.set_cookie instead of ngx.header["Set-Cookie"] works. The following code works now:

local cookies = {}
local args = {'test1', 'test2'}

for i=1, #args do
  cookies[i] = args[i] .. '=; expires=Thu, Jan 01 1970 00:00:00 UTC; domain=test.com;'
end

ngx.header.set_cookie = cookies

Can anyone explain the reasoning behind this behavior?

On Tuesday, January 12, 2016 at 4:57:55 PM UTC+5:30, Apoorv Saxena wrote:

Below code works fine:

ngx.header["Set-Cookie"] = {
   'test1=; expires=Thu, Jan 01 1970 00:00:00 UTC; domain=test.com;',
   'test2=; expires=Thu, Jan 01 1970 00:00:00 UTC; domain=test.com;'
}

though when trying to make the cookie names dynamic, it doesn't work:

local cookies = {}
local args = {'test1', 'test2'}

for i=1, #args do
  cookies[i] = args[i] .. '=; expires=Thu, Jan 01 1970 00:00:00 UTC; domain=test.com;'
end

ngx.header["Set-Cookie"] = cookies

have even tried using table.insert:

local cookies = {}
local args = {'test1', 'test2'}

for i=1, #args do
  table.insert(cookies, args[i] .. '=; expires=Thu, Jan 01 1970 00:00:00 UTC; domain=test.com;')
end

ngx.header["Set-Cookie"] = cookies

Issue seems with the assignment of variable to ngx.header["Set-Cookie"]

agentzh

Hello!

On Tue, Jan 12, 2016 at 3:27 AM, Apoorv Saxena wrote:
> though when trying to make the cookie names dynamic, it doesn't work:
>
>
> local cookies = {}
> local args = {'test1', 'test2'}
>
> for i=1, #args do
>   cookies[i] = args[i] .. '=; expires=Thu, Jan 01 1970 00:00:00 UTC;
> domain=test.com;'
> end
>
> ngx.header["Set-Cookie"] = cookies
>

Your example works for me with the following minimal configuration:

    location = /t {
        content_by_lua_block {
            local cookies = {}
            local args = {'test1', 'test2'}

            for i=1, #args do
              cookies[i] = args[i] .. '=; expires=Thu, Jan 01 1970
00:00:00 UTC; domain=test.com;'
            end

            ngx.header["Set-Cookie"] = cookies
        }
    }

Querying /t using curl:

    $ curl -i localhost:8080/t
    HTTP/1.1 200 OK
    Server: nginx/1.9.7
    Date: Tue, 12 Jan 2016 20:00:40 GMT
    Content-Type: text/plain
    Transfer-Encoding: chunked
    Connection: keep-alive
    Set-Cookie: test1=; expires=Thu, Jan 01 1970 00:00:00 UTC; domain=test.com;
    Set-Cookie: test2=; expires=Thu, Jan 01 1970 00:00:00 UTC; domain=test.com;

Maybe your OpenResty or ngx_lua is just too old? You can try the
latest OpenResty 1.9.7.2 RC1 version on your side.

Best regards,
-agentzh

asaxena001

Thanks agentzh, upgrade is pending for both openresty and ngx_lua.

On Wednesday, January 13, 2016 at 1:31:36 AM UTC+5:30, agentzh wrote:

Hello!

On Tue, Jan 12, 2016 at 3:27 AM, Apoorv Saxena wrote:
> though when trying to make the cookie names dynamic, it doesn't work:
>
>
> local cookies = {}
> local args = {'test1', 'test2'}
>
> for i=1, #args do
> cookies[i] = args[i] .. '=; expires=Thu, Jan 01 1970 00:00:00 UTC;
> domain=test.com;'
> end
>
> ngx.header["Set-Cookie"] = cookies
>

Your example works for me with the following minimal configuration:

location = /t {
content_by_lua_block {
local cookies = {}
local args = {'test1', 'test2'}

for i=1, #args do
cookies[i] = args[i] .. '=; expires=Thu, Jan 01 1970
00:00:00 UTC; domain=test.com;'
end

ngx.header["Set-Cookie"] = cookies
}
}

Querying /t using curl:

$ curl -i localhost:8080/t
HTTP/1.1 200 OK
Server: nginx/1.9.7
Date: Tue, 12 Jan 2016 20:00:40 GMT
Content-Type: text/plain
Transfer-Encoding: chunked
Connection: keep-alive
Set-Cookie: test1=; expires=Thu, Jan 01 1970 00:00:00 UTC; domain=test.com;
Set-Cookie: test2=; expires=Thu, Jan 01 1970 00:00:00 UTC; domain=test.com;

Maybe your OpenResty or ngx_lua is just too old? You can try the
latest OpenResty 1.9.7.2 RC1 version on your side.

Best regards,
-agentzh