Ji Bin

Results 45 comments of Ji Bin

@luisfmnunes could you try to fix the clang-format issue with `make clang-format` in the build directory? I could not directly merge your PR until you fix the format issue.

@luisfmnunes sorry for handling this late, could you rebase your branch, then the codecov may pass, as I just update the codecov patch coverage target.

Co-work with @chee003 From my perspective, the TLS handshake is handled by Lua already. So direct configuration with `ssl_stapling on;` may not work. apixis's nginx.conf ``` ssl_certificate_by_lua_block { apisix.stream_ssl_phase() }...

Outstanding jobs, 👍 I'll try do some local testing with the PR and updates here later.

Seems this **OCSP stapling** is already verified, great thanks to @yuweizzz this feature helped us a lot.

Looping @klesh @Startrekzky may also interested

> Does this support all gerrit versions? Not sure, we're using v3.4.3 and I've done some UT in code with the https://gerrit.onap.org/r (V3.7.2)

> The PR is currently marked as `DRAFT`, is it ready for review? `Apache-license-header` is still not solved, I'll fix it recently and then marked as **ready for review**.