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

daemon --ttl and other options should have range validation #56

Open
scott-ainsworth opened this issue Dec 6, 2021 · 0 comments · Fixed by #57
Open

daemon --ttl and other options should have range validation #56

scott-ainsworth opened this issue Dec 6, 2021 · 0 comments · Fixed by #57
Assignees
@scott-ainsworth
Labels
bug Something isn't working
Milestone
Version 1.1

Comments

@scott-ainsworth
Copy link
Owner

Problem

Currently the --polling-interval, --timeout, and --ttl options accept any u64 value.

Requirements

  • Add range validation for --polling-interval: 10 – 60 seconds.
  • Add range validation for --timeout: 10 – 300 seconds.
  • Add range validation for --ttl: 0 – 231-1 seconds.
@scott-ainsworth scott-ainsworth mentioned this issue Dec 6, 2021
Open
11 tasks
scott-ainsworth added a commit that referenced this issue Dec 6, 2021
@scott-ainsworth
Validate "seconds" command line options
8fc27ff 
- Validate the `--polling-interval` command line option
- Validate the `--timeout` command line option
- Validate the `--ttl` command line option

Issue #56
scott-ainsworth added a commit that referenced this issue Dec 6, 2021
@scott-ainsworth
Validate "seconds" command line options
db1f41f 
- Validate the `--polling-interval` command line option
- Validate the `--timeout` command line option
- Validate the `--ttl` command line option

Issue #56
@scott-ainsworth scott-ainsworth mentioned this issue Dec 7, 2021
Merged
@scott-ainsworth scott-ainsworth added the bug Something isn't working label Dec 7, 2021
@scott-ainsworth scott-ainsworth self-assigned this Dec 7, 2021
@scott-ainsworth scott-ainsworth added this to the Version 1.1 milestone Dec 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@scott-ainsworth scott-ainsworth

Labels
bug Something isn't working
Projects
None yet
Milestone
Version 1.1
Development

Successfully merging a pull request may close this issue.

Add connect timeout to daemon scott-ainsworth/avahi-aliases-rs
1 participant
@scott-ainsworth