fix bug in patten search; enable set in http scope; enable work with …

[zls0424]

• fix bug in patten search, use ngx_strlcasestrn instead of ngx_strcasestrn
• enable set in http scope
• enable work with userid module

[AirisX]

Hello @zls0424 ,

can you give an example of a case that leads to a bug that you try to fix in first commit?

[zls0424]

Hello @zls0424 ,

can you give an example of a case that leads to a bug that you try to fix in first commit?

Hi AirisX,

when a cookie with name uuid is set in response header, like

set-cookie: uuid=ZylrjlyJ2oAoAQksA1O5Ag==; expires=Thu, 31-Dec-37 23:55:55 GMT; path=/

then cookie names like uid, uuid, Ag set with set_cookie_flag will all match the cookie header line.

[edrandall]

What is the purpose of the 'do..while' loop here, the original code has no such loop?
It's hard to follow how the original code maps into the macro refactoring.
Personally I would avoid large #define's and refactor the repeated-code-blocks into a function call, then add a unit test.