Update default issuer helm value#2187
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughAdded a new optional JWT issuer configuration parameter ( Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2187 +/- ##
=======================================
Coverage 89.84% 89.85%
=======================================
Files 898 898
Lines 58937 58937
=======================================
+ Hits 52951 52955 +4
+ Misses 4420 4416 -4
Partials 1566 1566
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This pull request introduces a configuration option for the JWT issuer in the Helm chart values file. This allows users to optionally specify a custom issuer for JWT tokens, otherwise it will be derived from the server's URL.
Configuration improvements:
issuerfield under thejwtsection ininstall/helm/values.yamlto allow explicit configuration of the JWT issuer. If not set, it defaults to being derived from the server's URL.Summary by CodeRabbit