jinhua luo

Results 102 comments of jinhua luo

@monkeyDluffy6017 It's not for configuration, the long uri is used in the request. @caster-Q I could reproduce the error in my local env, and I would try to check if...

@caster-Q The route matching uses `pcre_exec` with jit enabled, but the long string matching exceeds the jit stack limit. APISIX sets the stack limit to a constant. When I increase...

@bungle According to my test, the following codes work: ```lua local session = require "resty.session".new({ storage = "shm" }) local session_id = session.encoder.encode(id) session.storage:destroy(session_id) ``` But such codes are tight...

Looks like some files are ignored. Let's check which files involve ldap: ```bash # cd to apisix repo dir git grep -i ldap ``` Not all files need to remove...

@Zhenye-Na Yes, you should look over all LDAP-related lines from the grep result and everything related to OpenLDAP (not LDAP) should be removed. Anyway, I'll review your output to ensure...

@Zhenye-Na Please rerun git grep and recheck the results when you're done making changes, i.e. no openldap-related files anymore, then I'll run CI again.

@Zhenye-Na Please check the CI errors.

> Acknowledged. > > Meanwhile, I am wondering is there a way that I could initiated the CI testing on my own to reduce the communication back and forth? >...

@Zhenye-Na forget to remove lualdap (which depends on openldap c lib) from rockspec? ``` Error: Failed installing dependency: https://luarocks.org/lualdap-1.2.6-1.src.rock - Could not find header file for LDAP No file ldap.h...

Why still compile lualdap? https://github.com/apache/apisix/actions/runs/6357262095/job/17274627068?pr=10176