Closed soapdog closed 2 years ago
Hi soapdog
I am using CGILua 6 for years and had no such problem. CGILua comes with its own launcher, cgilua.cgi
Anyway, I'll take a look at your patch.
Regards, Tomás
Tomás,
Thanks for the quick response. The reason for the patch was that sapi.lua
:
require "cgilua"
but the current source code doesn't appear to export cgilua.lua
or cgilua/init.lua
. I changed it to require main
directly by making it require "cgilua.main"
. I suspect that is some refactor in CGILua from some time ago.cgilua.main()
without parameters which caused it to fail some assertions here, I explicitly passed the parameters from wsapi.This enables it to work without requiring the cgilua launcher (which is actually from where I picked the needed code changes). It can work with Xavante and Sailor framework now.
Please, reach out if you want me to change anything. I made the minimal changes necessary to get it to work with the current CGILua.
Without these fixes, it doesn't work with Xavante...
There have been some changes in CGILua 6.x, this PR adds the missing parameters to
cgilua.main()
call in SAPI.