Exclude /metrics from LOGIN_REQUIRED regression #5746

Closed
opened 2025-12-29 19:32:08 +01:00 by adam · 2 comments
Owner

Originally created by @craized on GitHub (Dec 8, 2021).

Originally assigned to: @jeremystretch on GitHub.

NetBox version

v3.1.0

Python version

3.7

Steps to Reproduce

Set LOGIN_REQUIRED = True
Query /metrics endpoint

Expected Behavior

Receive metrics dump

Observed Behavior

Redirect to login page

Confirmed disabling LOGIN_REQUIRED allows fetching of /metrics
Reference issue for previous versions: https://github.com/netbox-community/netbox/issues/3123

Originally created by @craized on GitHub (Dec 8, 2021). Originally assigned to: @jeremystretch on GitHub. ### NetBox version v3.1.0 ### Python version 3.7 ### Steps to Reproduce Set LOGIN_REQUIRED = True Query /metrics endpoint ### Expected Behavior Receive metrics dump ### Observed Behavior Redirect to login page Confirmed disabling LOGIN_REQUIRED allows fetching of /metrics Reference issue for previous versions: https://github.com/netbox-community/netbox/issues/3123
adam added the type: bugstatus: accepted labels 2025-12-29 19:32:08 +01:00
adam closed this issue 2025-12-29 19:32:08 +01:00
Author
Owner

@jeremystretch commented on GitHub (Dec 8, 2021):

Seems that the URL provided by django_prometheus does not include a trailing slash.

@jeremystretch commented on GitHub (Dec 8, 2021): Seems that the URL provided by django_prometheus does not include a trailing slash.
Author
Owner

@craized commented on GitHub (Dec 8, 2021):

TYVM! Patch speed is second-to-none

@craized commented on GitHub (Dec 8, 2021): TYVM! Patch speed is second-to-none
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#5746