Search page opens up even is login should be required #7515

Closed
opened 2025-12-29 20:24:33 +01:00 by adam · 6 comments
Owner

Originally created by @abhi1693 on GitHub (Jan 14, 2023).

NetBox version

v3.4.2

Python version

3.10

Steps to Reproduce

Without logging in, go to http://localhost:8000/search/ and notice the page opens up

Expected Behavior

The page should redirect if login is required

Observed Behavior

Search page opens up
image

Originally created by @abhi1693 on GitHub (Jan 14, 2023). ### NetBox version v3.4.2 ### Python version 3.10 ### Steps to Reproduce Without logging in, go to http://localhost:8000/search/ and notice the page opens up ### Expected Behavior The page should redirect if login is required ### Observed Behavior Search page opens up ![image](https://user-images.githubusercontent.com/5083532/212461663-fc49980c-179e-4e85-ad58-8733456ca1a4.png)
adam added the type: bugstatus: revisions needed labels 2025-12-29 20:24:33 +01:00
adam closed this issue 2025-12-29 20:24:33 +01:00
Author
Owner

@abhi1693 commented on GitHub (Jan 14, 2023):

@jeremystretch If this is accepted, I'd like to take this

@abhi1693 commented on GitHub (Jan 14, 2023): @jeremystretch If this is accepted, I'd like to take this
Author
Owner

@jeremystretch commented on GitHub (Jan 18, 2023):

I'm not able to reproduce this on v3.4.2. With LOGIN_REQUIRED=True, visiting /search/ redirects me to /login/?next=/search/ as a non-authenticated user.

@jeremystretch commented on GitHub (Jan 18, 2023): I'm not able to reproduce this on v3.4.2. With `LOGIN_REQUIRED=True`, visiting `/search/` redirects me to `/login/?next=/search/` as a non-authenticated user.
Author
Owner

@abhi1693 commented on GitHub (Jan 18, 2023):

That's weird, I tried on the demo instance and I have the same issue. I believe login is required here

https://demo.netbox.dev/search/

@abhi1693 commented on GitHub (Jan 18, 2023): That's weird, I tried on the demo instance and I have the same issue. I believe login is required here https://demo.netbox.dev/search/
Author
Owner

@kkthxbye-code commented on GitHub (Jan 18, 2023):

I believe login is required here

It is not, when you go to the demo site you get the frontpage with all the locks. That is the behavior when LOGIN_REQUIRED is set to False and no EXEMPT_VIEW_PERMISSIONS are set.

@kkthxbye-code commented on GitHub (Jan 18, 2023): > I believe login is required here It is not, when you go to the demo site you get the frontpage with all the locks. That is the behavior when LOGIN_REQUIRED is set to False and no EXEMPT_VIEW_PERMISSIONS are set.
Author
Owner

@abhi1693 commented on GitHub (Jan 18, 2023):

I'll try this again tomorrow from my test instance. I may have made a mistake in the configuration that led to this page leak.

@abhi1693 commented on GitHub (Jan 18, 2023): I'll try this again tomorrow from my test instance. I may have made a mistake in the configuration that led to this page leak.
Author
Owner

@jeremystretch commented on GitHub (Jan 20, 2023):

Seems like this is likely a non-issue so I'm going to close it, but @abhi1693 if you find anything new please comment here and I'll re-open it. Thanks!

@jeremystretch commented on GitHub (Jan 20, 2023): Seems like this is likely a non-issue so I'm going to close it, but @abhi1693 if you find anything new please comment here and I'll re-open it. Thanks!
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#7515