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

change firewall recent.time to string as per VyOS 1.4 #313

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
6 changes: 4 additions & 2 deletions docs/vyos.vyos.vyos_firewall_rules_module.rst
Original file line number Diff line number Diff line change
Expand Up @@ -813,13 +813,15 @@ Parameters
<b>time</b>
<a class="ansibleOptionLink" href="#parameter-" title="Permalink to this option"></a>
<div style="font-size: small">
<span style="color: purple">integer</span>
<span style="color: purple">raw</span>
</div>
</td>
<td>
</td>
<td>
<div>Source addresses seen in the last N seconds.</div>
<div>Source addresses seen in the last time period.</div>
<div>Prior to 1.3, this is an integer representing a number of seconds.</div>
<div>After 1.3, this is a string of either &quot;second&quot;, &quot;minute&quot;, or &quot;hour&quot;.</div>
</td>
</tr>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -190,7 +190,7 @@ def __init__(self, **kwargs):
"recent": {
"options": {
"count": {"type": "int"},
"time": {"type": "int"},
"time": {"type": "str"},
},
"type": "dict",
},
Expand Down
6 changes: 4 additions & 2 deletions plugins/modules/vyos_firewall_rules.py
Original file line number Diff line number Diff line change
Expand Up @@ -293,8 +293,10 @@
type: int
time:
description:
- Source addresses seen in the last N seconds.
type: int
- Source addresses seen in the last time period.
- Prior to 1.3, this is an integer representing a number of seconds.
- After 1.3, this is a string of either "second", "minute", or "hour".
type: raw
source:
description:
- Source parameters.
Expand Down