Repository navigation
docs: sync tls.crl_file version annotation into TLS tables - #2733
Conversation
tls.crl_file is not present in v5.1.2 or any earlier tag. It landed on fluent-bit master after that release and ships in v5.1.3, so the tables that document it currently offer a setting no released build accepts. Append "Supported in v5.1.3 or later." to the tls.crl_file description in each table, matching the convention already used for tls.proxy.* in administration/transport-security.md. The reserved config key list in development/golang-output-plugins.md is left alone. It reserves a name rather than documenting availability. Refs fluent#2727 Signed-off-by: Eric D. Schabell <eric@schabell.org>
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (7)
Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review. 📝 WalkthroughWalkthroughThe ChangesCRL support version documentation
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~4 minutes Change: Other Merge Risk: ⚪ Minimal · up to The version note aligns with shared TLS support targeted for v5.1.3 and tells users when CRL revocation checks are available. No concrete regression is established, so the documentation change is ready for normal checks. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 2 systems. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@patrick-stephens ready for review! |
tls.crl_file is not present in v5.1.2 or any earlier tag. It landed on
fluent-bit master after that release and ships in v5.1.3, so the tables
that document it currently offer a setting no released build accepts.
Append "Supported in v5.1.3 or later." to the tls.crl_file description
in each table, matching the convention already used for tls.proxy.* in
administration/transport-security.md.
The reserved config key list in development/golang-output-plugins.md is
left alone. It reserves a name rather than documenting availability.
Refs #2727