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

Add support for DNSMASQ DHCP lease files #661

Closed
1 task done
derekoharrow opened this issue May 2, 2024 · 4 comments
Closed
1 task done

Add support for DNSMASQ DHCP lease files #661

derekoharrow opened this issue May 2, 2024 · 4 comments
Labels
Feature request➕ New feature or request next release/in dev image🚀 This is coming in the next release or was already released if the issue is Closed.

Comments

@derekoharrow
Copy link

Is there an existing issue for this?

  • I have searched the existing open and closed issues

Is your feature request related to a problem? Please describe

Tried using the dhcp plugin against the dnsmasq.leases file and it doesn't work.

Describe the solution you'd like

DNSmasq uses a different lease file format than DHCPd so I don't believe NetAlertX can interpret it.

Can you please add support for DNSmasq format?

The file is in the following format:

[Lease expiry time] [mac address] [ip address] [hostname] [client id, if known]

e.g.

1715932537 5c:87:9c:cf:97:02 192.168.1.115 ryans-laptop 01:5c:87:9c:cf:97:02

Describe alternatives you've considered

n/a

Anything else?

n/a

@derekoharrow derekoharrow added the Feature request➕ New feature or request label May 2, 2024
jokob-sk pushed a commit that referenced this issue May 2, 2024
@jokob-sk
Copy link
Owner

jokob-sk commented May 2, 2024

This should be available in the next release. If you can, please have a look at the netalertx-dev docker image, in about 15 minutes (or after the last action finishes) from now.

map your dnsmasq dhcp.leases file into a path that contains the string dnsmasq

It would be great if you could test this (backup everything first or use a new container) on your end by switching to the above image and letting me know if the issue was resolved.

Thanks in advance,
j

@derekoharrow
Copy link
Author

Wow! What took you so long? ;)

I'll give it a go now and report back. Thank you!!

@derekoharrow
Copy link
Author

Seems to be working great! Thank you!!

@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 May 2, 2024
@jokob-sk
Copy link
Owner

jokob-sk commented May 9, 2024

released -> closing for now

@jokob-sk jokob-sk closed this as completed May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature request➕ New feature or request 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