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

x-option for constant monitoring #35

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jiriatipteldotorg
Copy link
Contributor

introduce new monitoring mode that keeps sipsak running until
failureThis new -X mode keeps sending messages until a failure occurs. A
negative answer is not considered a failure. This is helpful when you
want to monitor a SIP device and do not want to restart sipsak all over
again — it keeps running until the device fails. In this mode, the
RFC3261 retransmission timers are ignored — they are constant.

Use it like this: keep trying every 1/4 sec and return with a failure
if 5 retries are not sufficient:
sipsak -s sip:[email protected] -X -Z 250 -D 5

I know it is not a best practice but I did several unnecessary changes
too — the code was simply too widely nested and I had a too narrow
monitor — I hope that’s fine. If not I can submit just the minimum
necessary changes instead.

introduce new monitoring mode that keeps sipsak running until
failureThis new -X mode keeps sending messages until a failure occurs. A
negative answer is not considered a failure. This is helpful when you
want to monitor a SIP device and do not want to restart sipsak all over
again — it keeps running until the device fails. In this mode, the
RFC3261 retransmission timers are ignored — they are constant.

Use it like this: keep trying every 1/4 sec and return with a failure
if 5 retries are not sufficient:
sipsak -s sip:[email protected] -X -Z 250 -D 5

I know it is not a best practice but I did several unnecessary changes
too — the code was simply too widely nested and I had a too narrow
monitor — I hope that’s fine. If not I can submit just the minimum
necessary changes instead.
Base automatically changed from master to main January 18, 2021 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant