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

Fixed an issue with the 500 error code on the Stream list page #4381

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

ZeroDeng01
Copy link

Fixed an issue with the 500 error code on the Streams listing page.

  • Since incoming_port is a numeric type stored in the database, it does not need a case when sorting.

close: #4302

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Fix stream list page error code 500。
@nginxproxymanagerci
Copy link

Docker Image for build 1 is available on
DockerHub
as nginxproxymanager/nginx-proxy-manager-dev:pr-4381

Note: ensure you backup your NPM instance before testing this image! Especially if there are database changes
Note: this is a different docker image namespace than the official image

@ZeroDeng01 ZeroDeng01 changed the title Fixed an issue with the 500 error code on the Streams listing page Fixed an issue with the 500 error code on the Stream list page Feb 22, 2025
@ZeroDeng01
Copy link
Author

@jc21 Available for review! THX!

@lengschder97
Copy link

@jc21 Available for review! THX!

Error is still present on my system after i applied the patch

@ZeroDeng01
Copy link
Author

@jc21 Available for review! THX!

Error is still present on my system after i applied the patch

Are you using sqlite or mysql? Can you provide more error information and logs? Are you using the docker image: nginxproxymanager/nginx-proxy-manager-dev:pr-4381? I use this image and everything works fine.

@fcaronte
Copy link

Yes the @ZeroDeng01 fix seem to work, I ask Gemini to make a script to can update from the proxmox community I'm using for ngixn lxc to your fork and then I can use stream. Yes I make a backup of everything and then I'll restore VM when a official update will come out but for now seem to work 💪🏻

Copy link

@fcaronte fcaronte left a comment

Choose a reason for hiding this comment

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

Work for me 👍🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Internal 500 Error on the stream list page
3 participants