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

Client with a "Client ID" is not shown in the query log #2607

Closed
ameshkov opened this issue Jan 27, 2021 · 1 comment
Closed

Client with a "Client ID" is not shown in the query log #2607

ameshkov opened this issue Jan 27, 2021 · 1 comment
Assignees
Milestone

Comments

@ameshkov
Copy link
Member

ameshkov commented Jan 27, 2021

Issue Details

  • Version of AdGuard Home server:
    • v0.105.0-SNAPSHOT-fc9ddcf9

I have two "Clients" configured:

  1. One for a "client ID" -- https://domain/dns-query/my-client -- let's call it "my-client"
  2. One for the IP address (for instance, 127.0.0.1) -- let's call it "local-client"

When a query is made to https://domain/dns-query/my-client from 127.0.0.1, AdGuard Home uses "my-client" client settings. However, in the query log, I see "local-client" instead.

Expected Behavior

I should see "my-client" in the query log.

Actual Behavior

I see "local-client" in the query log

I suppose we should mark clients/find as deprecated and add a new method that's better suited for this.

Instead of a list of IPs and client IDs, it should accept IP/ID pairs.

Moreover, I suppose the response format should be changed as well.

Here's what we're interested in:

  1. Client info (for manual clients)
  2. IP info (hostname, whois info, etc)
@ameshkov ameshkov added this to the v0.105.0 milestone Jan 27, 2021
heyxkhoa pushed a commit to heyxkhoa/AdGuardHome that referenced this issue Mar 20, 2023
Merge in DNS/adguard-home from 2607-querylog-client-id to master

Closes AdguardTeam#2607.

Squashed commit of the following:

commit 95367a8
Author: Ainar Garipov <A.Garipov@AdGuard.COM>
Date:   Thu Jan 28 19:02:02 2021 +0300

    all: fix client, imp docs

commit b652a7e
Author: Ainar Garipov <A.Garipov@AdGuard.COM>
Date:   Thu Jan 28 18:36:17 2021 +0300

    all: fix client id handling in querylog
@gpz1100
Copy link

gpz1100 commented Apr 29, 2024

I have added "https://adguard.domain.com/dns-query/abc123" to firefox (119.x) as the dns provider. Should it not include "abc123" somewhere in the visible query log? I can use "abc123" as a search key and it will return appropriate entries but with no search key entered the client column just shows the ip and hostname of the device.

Edit: Looks like the csv log contains a tag called "CID" where the client identifier is stored.

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

No branches or pull requests

3 participants