Skip to content

Update adguardhome-logs.yaml for parse new logs#1817

Open
prohand wants to merge 1 commit into
crowdsecurity:masterfrom
prohand:patch-1
Open

Update adguardhome-logs.yaml for parse new logs#1817
prohand wants to merge 1 commit into
crowdsecurity:masterfrom
prohand:patch-1

Conversation

@prohand

@prohand prohand commented Jun 3, 2026

Copy link
Copy Markdown

Description

The log format change to :

2026/06/03 21:13:32.653418 [error] webapi: http error host=dns.mondomaine.com method=POST url=/control/login status=403 ip=192.168.1.1 err="invalid username or password"

With this change the failed login is parsed

╭──────────────────────────────────────────────────────────╮
│ (Parser) LePresidente/adguardhome-logs │
├───────────────────────────────┬──────┬────────┬──────────┤
│ Parsers │ Hits │ Parsed │ Unparsed │
├───────────────────────────────┼──────┼────────┼──────────┤
│ file:/var/log/AdGuardHome.log │ 10 │ 8 │ 2 │

Checklist

  • [ x] I have read the contributing guide
  • [ x] I have tested my changes locally
  • For new parsers or scenarios, tests have been added
  • [ x] I have run the hub linter and no issues were reported (see contributing guide)
  • Automated tests are passing
  • [x ] AI was used to generate any/all content of this PR

The log format change to : 

2026/06/03 21:13:32.653418 [error] webapi: http error host=dns.mondomaine.com method=POST url=/control/login status=403 ip=192.168.1.1 err="invalid username or password"
@sabban

sabban commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

hi,

Thanks to your contribution.

This should include some tests as well.

Regards,

@sabban sabban self-assigned this Jun 8, 2026
@sabban sabban added the waiting-for-user Waiting for submitter answer regarding specific points in the PR label Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

waiting-for-user Waiting for submitter answer regarding specific points in the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants