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

store is nil #7

Open
linux2014linux opened this issue Mar 11, 2021 · 5 comments
Open

store is nil #7

linux2014linux opened this issue Mar 11, 2021 · 5 comments

Comments

@linux2014linux
Copy link

linux2014linux commented Mar 11, 2021

Cas login pass, cas.lua error happen:

2021/03/11 17:49:04 [error] 87487#2288134: *2 lua entry thread aborted: runtime error: /usr/local/Cellar/openresty/1.19.3.1_1/lualib/cas.lua:78: attempt to index upvalue 'store' (a nil value)
stack traceback:
coroutine 0:
        /usr/local/Cellar/openresty/1.19.3.1_1/lualib/cas.lua: in function '_set_store_and_cookie'
        /usr/local/Cellar/openresty/1.19.3.1_1/lualib/cas.lua:118: in function 'validate_with_CAS'
        /usr/local/Cellar/openresty/1.19.3.1_1/lualib/cas.lua:135: in function 'forceAuthentication'
        access_by_lua(nginx.conf:35):1: in main chunk, client: 127.0.0.1, server: console.cloud.ll.com, request: "GET /s1?ticket=ST-5-9Z-euig-7AtF0nZnLrKVZTeebc0liulei07 HTTP/1.1", host: "console.cloud.ll.com"
prigaux added a commit that referenced this issue Mar 11, 2021
@prigaux
Copy link
Member

prigaux commented Mar 11, 2021

I have enhanced error message for this configuration issue.

@linux2014linux
Copy link
Author

New error:

writing a global Lua variable ('store_name') which may lead to race conditions between concurrent requests, so prefer the use of 'local' variables

prigaux added a commit that referenced this issue Mar 11, 2021
@prigaux
Copy link
Member

prigaux commented Mar 11, 2021

Oops, my version of nginx+lua did not warn me :-(

Corrected in updated commit.

@linux2014linux
Copy link
Author

@prigaux The cas.lua basic logci may be ok, but it has many lua grammar error.

@prigaux
Copy link
Member

prigaux commented Mar 11, 2021

It has been tested and validated with debian 10 libnginx-mod-http-lua which uses lua nginx 0.10.13 (ref)

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

No branches or pull requests

2 participants