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 vendor listings to DHCP lease table #2605
base: devel
Are you sure you want to change the base?
Conversation
Thanks for the PR. There are some tests failing. |
Yeah I was editing this on my home server through a poorly-configured vim - fixing those lint issues. For v6, should I avoid trying to merge into |
Yes. We are still trying to implement (more or less) everything we had with v5 and fix the biggest issues.
You can already rebase on You'll notice, we try to get rid of |
3b149ca
to
5e2a6e1
Compare
Not everything has a hostname; this helps determine what is what when looking through DHCP leases. Uses the existing macvendor SQLite table that FTL obtains from the Wireshark repo. Signed-off-by: Jacob Masen-Smith <smith.jacob@gmail.com>
5e2a6e1
to
f4b4a4d
Compare
@yubiuser I hate to ask here but couldn't find an appropriate location, even in Discourse - is there documentation on the new API paradigm? Looking at devel-v6 it's hard to determine where the endpoints the Javascript AJAX calls are. I tried looking at FTL as well and wasn't able to tell. |
@smithjacobj The idea is that FTL ships the documentation for its API on its won instead of hosting it online. This has the benefit that different branches will always have an up-to-date documentation as the CI precisely checks if all endpoints are documented, if the documentation contains an endpoint which doesn't even exist, if the endpoints have the exact structure as written in the documentation, if the examples in the documentation follow this exact structure, ... For now, you can see this documentation where the v6.0 FTL code lives, in the future it will also be uploaded to a less hidden place for someone wanting to checkout the docs before installing FTL or for those wanting to read the https://ftl.pi-hole.net/new/http/docs This documentation is always automatically updated after each successful CI testing of FTL. PS: The logo and favicon are missing is an artifact of the current location of the docs. I didn't want to compile them into FTL. When using the self-hosted docs on your Pi-hole, the images offered by the web interface are used and you can see them. |
@DL6ER thanks, I was looking at the wrong FTL branch. It looks like I should be looking at new/http instead of development? |
Yes, sorry. It should be |
Thank you for your contribution to the Pi-hole Community!
Please read the comments below to help us consider your Pull Request.
We are all volunteers and completing the process outlined will help us review your commits quicker.
Please make sure you
What does this PR aim to accomplish?:
Not everything has a hostname; this adds a column containing the vendor to the DHCP leases table to help determine what is what when looking through DHCP leases.
How does this PR accomplish the above?:
This change uses the existing
macvendor
SQLite table that FTL obtains from the Wireshark repo.Link documentation PRs if any are needed to support this PR:
N/A?
By submitting this pull request, I confirm the following:
git rebase
)