Skip to content

fix: disable profiling endpoint by default - #232

Merged
petersbingham merged 1 commit into
mainfrom
task/sec-disable-profiling
Sep 10, 2026
Merged

petersbingham merged 1 commit into
mainfrom
task/sec-disable-profiling

Conversation

@petersbingham

Copy link
Copy Markdown
Contributor

Summary

  • Sets profiling: false in charts/registry/values.yaml default values
  • The pprof profiling endpoint was inadvertently enabled on the status server in all environments with no production override

Identified as a widespread pattern across openkcm repos — same fix being applied to extauthz and platform-extension-service.

Test plan

  • No functional change to the main application — status server profiling endpoint only

@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 5a2586cc-5537-45b2-aaef-3cc25de2c61b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cb80 cb80 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also bump the chart version in charts/registry/Chart.yaml

protectorT1
protectorT1 previously approved these changes Sep 8, 2026
@petersbingham
petersbingham force-pushed the task/sec-disable-profiling branch from f30d51c to dad9780 Compare September 10, 2026 11:01
Signed-off-by: Peter Bingham <petersbingham@hotmail.co.uk>
@petersbingham
petersbingham force-pushed the task/sec-disable-profiling branch from dad9780 to f421b74 Compare September 10, 2026 11:04
@petersbingham
petersbingham merged commit 8dff992 into main Sep 10, 2026
11 checks passed
@petersbingham
petersbingham deleted the task/sec-disable-profiling branch September 10, 2026 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants