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.

Sign up for GitHub

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

Jump to bottom

Client shows 0.0.0.0 #1329

Closed
cruunnerr opened this issue Jan 4, 2020 · 4 comments
Closed

Client shows 0.0.0.0 #1329

cruunnerr opened this issue Jan 4, 2020 · 4 comments
Assignees
@szolin
Labels
bug P3: Medium
Milestone
v0.100 patch

Comments

@cruunnerr
Copy link

cruunnerr commented Jan 4, 2020 •
edited
Loading

Hi guys,

I use the actual Version of AdGuard Home. Since i updated last time I have multiple Query logs show "0.0.0.0" as Client address.

I captured the NIC of my Router to see what happens there... So the client with the IP "10.10.20.203" sends DNS request to my AdGuard Home Server (10.10.10.10).
Adguard sends to the client the IP "0.0.0.0" and in the Web Interface AdGuard shows me, that the Client-Address would be "0.0.0.0"....

I don't understand that.... Sorry guys. Not the best IT-specialist ^^

Bildschirmfoto 2020-01-04 um 10 10 00

Bildschirmfoto 2020-01-04 um 10 10 16

Edit:
Oh, and it's just a normal Filter-List:
https://raw.githubusercontent.com/cruunnerr/Adguard/master/Blocklist.txt
@nelox
Copy link

nelox commented Jan 6, 2020

I have multiple Query logs show "0.0.0.0" as Client address.

I observe the same behaviour (AdGuardHome v0.100.8):

Screen Shot 2020-01-06 at 19 19 02

Filter list:
https://raw.githubusercontent.com/StevenBlack/hosts/master/hosts

@htjain
Copy link

htjain commented Jan 6, 2020

+1

@ameshkov ameshkov added this to the v0.100 patch milestone Jan 6, 2020
@Mosney
Copy link

Mosney commented Jan 7, 2020

ditto, rule is https://cdn.jsdelivr.net/gh/neoFelhz/neohosts@gh-pages/full/hosts.txt
image

@szolin
Copy link
Contributor

szolin commented Jan 9, 2020

Yes, thank you!
A quick fix is coming...
A normal fix will be in #882 where we will stop using "Result"{} json object in logs.

@szolin szolin mentioned this issue Jan 14, 2020
Closed
adguard pushed a commit that referenced this issue Oct 19, 2021
@105th
Pull request #1329: 3529 validate dhcpv4
d7aafa7 
Merge in DNS/adguard-home from 3529-validate-dhcpv4 to master

Squashed commit of the following:

commit 2f2455a
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Tue Oct 19 19:18:12 2021 +0300

    dhcpv4: better test && fix changelog

commit ec4ff91
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 19:08:44 2021 +0300

    dhcpv4: better tests

commit e0e2f27
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 18:55:47 2021 +0300

    dhcpv4: better tests

commit 73e1d08
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 18:47:21 2021 +0300

    dhcpv4: better tests

commit f636fc3
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 18:47:07 2021 +0300

    all: updated golibs

commit 86dd107
Merge: 51aaf6d b296fa2
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 17:18:17 2021 +0300

    Merge branch 'master' into 3529-validate-dhcpv4

commit 51aaf6d
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 17:18:02 2021 +0300

    dhcpv4: better changelog

commit 720b896
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 17:14:25 2021 +0300

    dhcpv4: better tests

commit 1098bef
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 12:08:26 2021 +0300

    dhcp: changelog

commit d1f6c89
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 12:03:06 2021 +0300

    dhcpv4: fixed tests

commit 8b67134
Merge: 9991e9c 3fa38fb
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 11:57:57 2021 +0300

    Merge branch 'master' into 3529-validate-dhcpv4

commit 9991e9c
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 11:55:40 2021 +0300

    dhcpv4: added tests

commit 5798a80
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 11:46:03 2021 +0300

    dhcpv4: validate subnet mask and ip range
heyxkhoa pushed a commit to heyxkhoa/AdGuardHome that referenced this issue Mar 20, 2023
@105th @heyxkhoa
Pull request AdguardTeam#1329: 3529 validate dhcpv4
28ffc35 
Merge in DNS/adguard-home from 3529-validate-dhcpv4 to master

Squashed commit of the following:

commit 2f2455a
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Tue Oct 19 19:18:12 2021 +0300

    dhcpv4: better test && fix changelog

commit ec4ff91
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 19:08:44 2021 +0300

    dhcpv4: better tests

commit e0e2f27
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 18:55:47 2021 +0300

    dhcpv4: better tests

commit 73e1d08
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 18:47:21 2021 +0300

    dhcpv4: better tests

commit f636fc3
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 18:47:07 2021 +0300

    all: updated golibs

commit 86dd107
Merge: 51aaf6d b296fa2
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 17:18:17 2021 +0300

    Merge branch 'master' into 3529-validate-dhcpv4

commit 51aaf6d
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 17:18:02 2021 +0300

    dhcpv4: better changelog

commit 720b896
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 17:14:25 2021 +0300

    dhcpv4: better tests

commit 1098bef
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 12:08:26 2021 +0300

    dhcp: changelog

commit d1f6c89
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 12:03:06 2021 +0300

    dhcpv4: fixed tests

commit 8b67134
Merge: 9991e9c 3fa38fb
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 11:57:57 2021 +0300

    Merge branch 'master' into 3529-validate-dhcpv4

commit 9991e9c
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 11:55:40 2021 +0300

    dhcpv4: added tests

commit 5798a80
Author: Dmitriy Seregin <d.seregin@adguard.com>
Date:   Mon Oct 18 11:46:03 2021 +0300

    dhcpv4: validate subnet mask and ip range
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@szolin szolin

Labels
bug P3: Medium
Projects
None yet
Milestone
v0.100 patch
Development

No branches or pull requests
6 participants
@nelox @ameshkov @cruunnerr @Mosney @szolin @htjain