Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

owner is null #965

Closed
musgos opened this issue Jul 22, 2024 · 10 comments
Closed

owner is null #965

musgos opened this issue Jul 22, 2024 · 10 comments

Comments

@musgos
Copy link

musgos commented Jul 22, 2024

after deleting admin user can´t see anything.
All is "owner is
image

@Zoey2936
Copy link
Member

so this happens if you delete the user, which created the host?

@musgos
Copy link
Author

musgos commented Jul 22, 2024

create 2 new admin users, log off, login with new user, delete admin user. log off -> login and "owner is null"
i think it is because in DB the creator of the proxy hosts was admin, but i'm guessing

@musgos
Copy link
Author

musgos commented Jul 22, 2024

so this happens if you delete the user, which created the host?

yes, probably

@musgos
Copy link
Author

musgos commented Jul 22, 2024

If i create a new proxy host, cant see it in the list (same message) , but it i apply a filter (in search host...) it will appear, then gone after removal of filter

PS every proxy is working normal

@JSouthGB
Copy link

JSouthGB commented Jul 24, 2024

I suspect this is related to an upstream issue. If it is, connecting to the db and changing the is_deleted flag to 0 would correct it. Or maybe needing to change the unique id to 1.

For reference, NginxProxyManager#230 - my findings

@Zoey2936
Copy link
Member

I must say, I think I can't fix this, but PR are welcome (you can also create one in upstream). Otherwise it should workl in the php version (in the future), since in the first time there will be no multiple user support

@JSouthGB
Copy link

I must say, I think I can't fix this, but PR are welcome (you can also create one in upstream). Otherwise it should workl in the php version (in the future), since in the first time there will be no multiple user support

Due to issues like this as well as others, I've moved on from NPM. I had tried this fork, but found some upstream issues still present. Also, I've never implemented multiple users in NPM, but these issues still occurred. Once the rewrite is complete, I'll probably circle back to check it out.

I'm also nowhere near knowledgeable enough to even begin opening a PR.

@Zoey2936
Copy link
Member

there is PR in upstream: https://github.com/NginxProxyManager/nginx-proxy-manager/pull/4166/files

@JSouthGB
Copy link

there is PR in upstream: https://github.com/NginxProxyManager/nginx-proxy-manager/pull/4166/files

Great news! Looks like it was a fairly straightforward fix.

Zoey2936 added a commit that referenced this issue Nov 16, 2024
by merging https://github.com/NginxProxyManager/nginx-proxy-manager/pull/4166/files and adding multi lang support

Co-Authored-By: Remco Kersten <[email protected]>
Zoey2936 added a commit that referenced this issue Nov 16, 2024
@Zoey2936
Copy link
Member

can you please try the develop tag if this fixes it?

Zoey2936 added a commit that referenced this issue Nov 17, 2024
Zoey2936 added a commit that referenced this issue Nov 21, 2024
Zoey2936 added a commit that referenced this issue Nov 21, 2024
Zoey2936 added a commit that referenced this issue Nov 21, 2024
Zoey2936 added a commit that referenced this issue Nov 22, 2024
Zoey2936 added a commit that referenced this issue Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants