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

Sorting of iP addresses not correct #983

Closed
4 of 7 tasks
Stationary-1 opened this issue Jan 29, 2025 · 3 comments
Closed
4 of 7 tasks

Sorting of iP addresses not correct #983

Stationary-1 opened this issue Jan 29, 2025 · 3 comments
Labels
bug 🐛 Something isn't working next release/in dev image🚀 This is coming in the next release or was already released if the issue is Closed.

Comments

@Stationary-1
Copy link

Stationary-1 commented Jan 29, 2025

Is there an existing issue for this?

The issue occurs in the following browsers. Select at least 2.

  • Firefox
  • Chrome
  • Edge
  • Safari (unsupported) - PRs welcome
  • N/A - This is an issue with the backend

Current Behavior

The system does not sort the IP-Addresses in increasing order, but like . 1, .10, .100, .2, .20, .200, .3, .30, .4, .40 etc in the connected device list, when sorting by IP.
Image

Expected Behavior

Sort the IP in increasing number .1, .2, .3, .4, .10, .20, .30, .40, .100, .200 etc

Steps To Reproduce

Click on Network, Under connected devices click to sort by IP address.

Version 25.1.19

app.conf

same as last time

docker-compose.yml

same as last time

What installation are you running?

Production (netalertx)

app.log

It is not really a bug in the true sense, where something is going wrong. I believe the system is parsing the IP-Address digit by digit when sorting them, instead of going for the numbers.

Debug enabled

  • I have read and followed the steps in the wiki link above and provided the required debug logs and the log section covers the time when the issue occurs.
@Stationary-1 Stationary-1 added the bug 🐛 Something isn't working label Jan 29, 2025
jokob-sk pushed a commit that referenced this issue Feb 1, 2025
@jokob-sk
Copy link
Owner

jokob-sk commented Feb 1, 2025

This should be available in the next release. It would be great if you could test this. Can you please switch to the netalertx-dev docker image (backup everything at first or, ideally spin up a separate container with a separate db and config), in about 15 minutes (or after the last action finishes) from now?

Make sure you refresh your browser cache - and click the 🔄 refresh button in the top right corner.

Consider sharing the app on these socials 🙂🙏

Thanks in advance,
j

@jokob-sk jokob-sk added the next release/in dev image🚀 This is coming in the next release or was already released if the issue is Closed. label Feb 1, 2025
@Stationary-1
Copy link
Author

Stationary-1 commented Feb 1, 2025

👍in the dev Version it works as expected, sorted by increasing number.

@jokob-sk
Copy link
Owner

jokob-sk commented Feb 8, 2025

releasing in 15 min -> closing

@jokob-sk jokob-sk closed this as completed Feb 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working next release/in dev image🚀 This is coming in the next release or was already released if the issue is Closed.
Projects
None yet
Development

No branches or pull requests

2 participants