chore: bump apisix-nginx-module to 1.19.10 - #489
Conversation
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 13 minutes Limit details: You’ve used the included review currently available. Your 60 included PR review attempts over the past 7 days set your current allowance at 1 review per hour. You’re in a promotional period — use the checkbox below to run this review for free:
On-demand reviews are free for the next 30 days. After that, they cost $0.25 per reviewed file. How can I continue?Run this review now using the option above, or comment You can also wait for the limit to reset, then comment An organization admin can change what happens after included review limits in Billing. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
Comment |
Picks up api7/apisix-nginx-module#126, which makes
upstream.set_ssl_trusted_store()take effect without a client certificate.Before that fix the trusted store was applied inside the
ctx->upstream_cert != NULLbranch ofngx_http_apisix_set_upstream_ssl(), so a caller that turns verification on at runtime and supplies its own CA — the no-mTLS case — ended up with an empty trust store and failed every handshake with(20:unable to get local issuer certificate).APISIX needs it for
upstream.tls.ca_certs(apache/apisix#13863); a runtime release will follow this merge.