Allow to `disable_protection` for a specific duration

[rbeuque74]

Problem/Motivation

Since AGH v0.107.28, AGH allow to disable protection for a specific duration (see AdguardTeam/AdGuardHome#1333).
This option is not present in disable_protection, hence, cannot be used in Home Assistant, using a service.

Expected behavior

Having an optional pause_duration_miliseconds parameter.

Actual behavior

None

Proposed changes

• switch URL to POST /protection
• add an optional parameter to disable_protection

Refers to #1030 #1336, code ready was proposed in closed PR #1033

cc @dbensmith

[github-actions]

There hasn't been any activity on this issue recently, so we clean up some of the older and inactive issues.
Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by leaving a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thanks!

[dbensmith]

Still need this!

[rbeuque74]

Still need this!

[olli991]

Also would like to push this feature. Would be pretty neat to use the features within Home Assistant.

Looking into the Adguard OpenAPI Changelog the options seem to be in there:
https://github.com/AdguardTeam/AdGuardHome/blob/21834ee3c5b5707ef1f26822697a0af946529663/openapi/CHANGELOG.md#new-protection_disabled_until-field-in-get-controldns_info-response

So we could get the info and also could call the timeout it seems

[github-actions]

There hasn't been any activity on this issue recently, so we clean up some of the older and inactive issues.
Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by leaving a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thanks!

[rbeuque74]

Not stale, multiple people want this, code is ready, just waiting for a review 🤗

[dbensmith]

@rbeuque74 it might get more visibility if you resubmitted the PR for review. Would it be too much work to try that?

[rbeuque74]

@dbensmith Code is ready, I did that the first time but it didn't help. After I read the contributors guidelines that ask for an issue before opening the pull request so that's what I did.

I thought one way of doing it could be to open an issue on home-assistant/core to have that feature introduced, that would led to the patch of this lib. But never gave it a try

Are you familiar with the home-assistant community so we can do that ?

[github-actions]

There hasn't been any activity on this issue recently, so we clean up some of the older and inactive issues.
Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by leaving a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thanks!

[rbeuque74]

Hello,
Still hoping getting this one merged!