AdguardTeam / AdguardTeam/AdGuardHome

Do not show 'port 443 is not available' error notification if encryption is not enabled (again)

Đang mở
#2,407 3 bình luận 0 reaction 0 người được giao Xem trên GitHub
bug P3: Medium UI
Ngôn ngữ chính
TypeScript
Star
36.9k
Fork
2.5k
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

Steps to reproduce

Install AdGuard Home on a system where port 443 is already in use.
Open AdGuard Home settings page.
Observe the red error notification with the message "Error: control/tls/validate | port 443 is not available, cannot enable HTTPS on it | 400".

Expected behavior

If "Enable Encryption" option is disabled, do not show any error notifications related to unavailable ports specified on this section.

Actual behavior

If "Enable Encryption" option is disabled, error notifications related to unavailable ports specified on this section are still shown even though the option is disabled and not being used.

Screenshot:
![imagen](https://user-images.githubusercontent.com/30116095/101559195-edbca780-39c0-11eb-81ac-bdac22df1129.png)

Your environment

Description | Value
-- | --
Version of AdGuard Home server: | (v0.104.1)

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.