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

Show which IP addrs are duplicated in error messages #3975

Closed
Eyeborgs opened this issue Dec 21, 2021 · 4 comments
Closed

Show which IP addrs are duplicated in error messages #3975

Eyeborgs opened this issue Dec 21, 2021 · 4 comments
Assignees
@ainar-g
Labels
enhancement P3: Medium
Milestone
v0.107.1

Comments

@Eyeborgs
Copy link

  • Version of AdGuard Home server:
    • v0.107.0
  • How did you install AdGuard Home:
    • GitHub releases
  • How did you setup DNS configuration:
    • System
  • Operating system and version:
    • Windows 10, Windows Server

Screenshots

Screenshot:

image

Вы можете оставьте проверку дубликатов и выводить уведомления но при этом чтоб была возможность все равно сохранять сохранения в "Запрещённые клиенты" потому что у меня заблокированных IP 9311 в CIDR диапазоне но при этом не все они в CIDR диапазоне и скорее всего что то там не позволяет это сделать а искать это не реально..
В v.106.3 такого не было и все работало как нужно.. Временно возвращаюсь на v.106.3
@Eyeborgs
Copy link
Author

Eyeborgs commented Dec 21, 2021 •
edited
Loading

Ну или в Крайнем случае пускай уведомляют какие IP конфликтуют..

@ainar-g ainar-g changed the title Error: control/access/set | duplicates in disallowed clients | 400 Show which IP addrs are duplicated in error messages Dec 21, 2021
@ainar-g ainar-g added this to the v0.107.1 milestone Dec 21, 2021
@ainar-g ainar-g self-assigned this Dec 21, 2021
@ainar-g ainar-g mentioned this issue Dec 24, 2021
Closed
adguard pushed a commit that referenced this issue Dec 28, 2021
@ainar-g
Pull request: all: imp uniq validation err msgs
d2ce06e 
Updates #3975.

Squashed commit of the following:

commit f8578c2
Author: Ainar Garipov <A.Garipov@AdGuard.COM>
Date:   Tue Dec 28 16:39:13 2021 +0300

    aghalgo: imp docs

commit d9fc625
Author: Ainar Garipov <A.Garipov@AdGuard.COM>
Date:   Tue Dec 28 16:21:24 2021 +0300

    all: imp uniq validation err msgs
@ainar-g
Copy link
Contributor

ainar-g commented Dec 28, 2021

@Eyeborgs, на канале edge мы улучшили валидациционное сообщение:

Screenshot from 2021-12-28 16-17-08

Вы можете протестировать, работает ли это в вашем случае? Или можете подождать релиза, так как релиз 0.107.1 должен выйти в ближайшие дни.

@Eyeborgs
Copy link
Author

@ainar-g Протестировал и вроде как работает нашел три дубликата но странно то что я удалил один дублирующий IP и сохранил настройки а два оставил и он перестал выводить ошибку что есть дубликаты и просто успешно все сохранял - такое ощущение что после первого удаленного дубликата эта система просто переставала работать ну проверять и просто сохраняла значения

@ainar-g
Copy link
Contributor

ainar-g commented Dec 29, 2021

@Eyeborgs, спасибо за проверку!

странно то что я удалил один дублирующий IP и сохранил настройки а два оставил и он перестал выводить ошибку что есть дубликаты и просто успешно все сохранял

Мы не можем это воспроизвести, к сожалению. Если у вас эта проблема стабильно воспроизводится, откройте, пожалуйста, новый тикет с описанием шагов.

@ainar-g ainar-g closed this as completed Dec 29, 2021
adguard pushed a commit that referenced this issue Dec 29, 2021
@ainar-g
cherry-pick: all: imp uniq validation err msgs
74dcc91 
Updates #3975.

Squashed commit of the following:

commit f8578c2
Author: Ainar Garipov <A.Garipov@AdGuard.COM>
Date:   Tue Dec 28 16:39:13 2021 +0300

    aghalgo: imp docs

commit d9fc625
Author: Ainar Garipov <A.Garipov@AdGuard.COM>
Date:   Tue Dec 28 16:21:24 2021 +0300

    all: imp uniq validation err msgs
heyxkhoa pushed a commit to heyxkhoa/AdGuardHome that referenced this issue Mar 20, 2023
@ainar-g @heyxkhoa
Pull request: all: imp uniq validation err msgs
d2ad44b 
Updates AdguardTeam#3975.

Squashed commit of the following:

commit f8578c2
Author: Ainar Garipov <A.Garipov@AdGuard.COM>
Date:   Tue Dec 28 16:39:13 2021 +0300

    aghalgo: imp docs

commit d9fc625
Author: Ainar Garipov <A.Garipov@AdGuard.COM>
Date:   Tue Dec 28 16:21:24 2021 +0300

    all: imp uniq validation err msgs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ainar-g ainar-g

Labels
enhancement P3: Medium
Projects
None yet
Milestone
v0.107.1
Development

No branches or pull requests
2 participants
@ainar-g @Eyeborgs