fix: preserve full controller path

[alexplatteeuw]

Background

A recent change in this commit altered how the controller name is recorded.

Previously, metrics used params["controller"], which returns the full namespaced path:

controller = params["controller"]
# => "authentication/sessions"

After the commit, we switched to using controller_instance.controller_name, which only returns the base controller name, stripping namespaces:

controller = controller_instance.controller_name
# => "sessions"

Problem

This change breaks existing queries on our side that rely on the full path. For example:

http_requests_total{controller=~"authentication/.*"}

no longer matches because controller_name only provides "sessions" instead of "authentication/sessions".

Solution

Use controller_instance.controller_path, which preserves the previous behavior:

controller = controller_instance.controller_path
# => "authentication/sessions"

This restores compatibility with existing queries.

[n-rodriguez]

LGTM

[flori]

Now you just have to release it with a new version, then our api/foobar metrics would reappear in our grafana after the had suddenly disappeared.

[SamSaffron]

releasing a new version today

…

On Tue, Nov 11, 2025 at 12:14 AM Florian Frank ***@***.***> wrote: *flori* left a comment (discourse/prometheus_exporter#353) <#353 (comment)> Now you just have to release it with a new version, then our api/foobar metrics would reappear in our grafana after the had suddenly disappeared. — Reply to this email directly, view it on GitHub <#353 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAABIXIEHNW7ZEHJ7QBA4HD34CFTJAVCNFSM6AAAAACE2ISDDGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTKMJRGU3TINJYGI> . You are receiving this because you modified the open/close state.Message ID: ***@***.***>