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

autopatch updates with IAX/SIP notes #97

Merged
merged 2 commits into from
Jan 11, 2025
Merged

autopatch updates with IAX/SIP notes #97

merged 2 commits into from
Jan 11, 2025

Conversation

knobby4444
Copy link
Collaborator

Major step up in autopatch documentation. Added Mason's github info on voip.ms IAX setup and my own flowroute SIP setup info. Other updates include duplex info.

@Allan-N Allan-N assigned Allan-N and knobby4444 and unassigned Allan-N Dec 24, 2024
@Allan-N Allan-N requested review from KB4MDD, jxmx and Allan-N December 24, 2024 15:25
Major step up in autopatch documentation.  Added Mason's github info on voip.ms IAX setup and my own flowroute SIP setup info. Using general setup info instead of specific info for voip.ms and flowroute.com, although there is still some specific info in the configs that I think is helpful to leave in there.
Copy link
Collaborator

@Allan-N Allan-N left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

General things to look at :

  • Sometimes you refer to the file (rpt.conf) and other times the full path (/etc/asterisk/rpt.conf).
  • I noticed that sometimes you refer to "IAX and PJSIP" and other times "PJSIP and IAX".
  • It would be good to use a spell checker on the entire file

This guide includes example configuration files that may need adjustments to get
working with specific providers and ASL versions.

This guide will document the general process for setting up IAX2 and SIP providers with expamples for voip.ms IAX and flowroute.com SIP. Any provider should be setup in a similar fashion.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

expamples --> examples

docs/adv-topics/autopatch.md Show resolved Hide resolved
```

In the **rpt.conf** configuration file, a context in extensions.conf is defined
### rpt.conf
In the **/etc/asterisk/rpt.conf** configuration file, a context in extensions.conf is defined
in a node stanza using the context key:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm trying to process this sentence. I think what you are trying to write is that one needs to setup a context in the /etc/asterisk/extensions.conf file. Then, in the /etc/asterisk/rpt.conf file one needs to reference that context in a node stanza.

@@ -66,64 +161,279 @@ context = autopatch ; dialing context for phone
callerid = "Repeater" <0000000000> ; callerid for phone calls
accountcode = RADIO ; account code (optional)
```
Also in the **rpt.conf** file, entries are added to the function stanza so
Also in the **/etc/asterisk/rpt.conf** file, entries are added to the function stanza so
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not wild about "Also in the ...". Maybe:

In the rpt.conf file, entries should also be added to the function stanza so ...

| exten | Overrides the default autopatch extension. |
| nostar | Disables the repeater function prefix |

#### Create a Script to Handle Terminating Calls
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this script work if exec'd as the "asterisk" user (as would happen with the "0 = cmd,..." function you detailed above)?

#!/bin/bash
# Target pattern to match (adjust as needed for different calls)
TARGET_PATTERN="IAX2/voipms-"
# TARGET_PATTERN="PJSIP/flowroute-"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any way to avoid having to specify explicit patterns here (and below where the patterns are checked)? I'm thinking that any "PJSIP/" can be targeted. The question is whether we can tell the difference from an IAX2 "ASL" connection and an IAX2 "phone" connection.

```
For outbound SIP connections it should look like this:

### extentions.conf
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

extentions.conf --> extensions.conf

... and do you want to use the full path?

In **/etc/asterisk/extensions.conf**, a stanza for the autopatch context is added which
refers to the peer stanza in iax.conf or pjsip.conf. These examples will likely need to be adjusted to work with a specific provider:

For outbound IAX connections extensions.conf:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For outbound IAX connections, add :

exten => _NXXNXXXXXX,n,Hangup()

```
For outbound SIP connections extensions.conf, this example uses IP auth:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For outbound SIP connections using IP auth, add :

exten => _NXXNXXXXXX,n,Hangup()
```

Setup 911 extention
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To setup the 911 extension, add :

(and yes, here you also need to correct the spelling of extension :-)

@knobby4444
Copy link
Collaborator Author

Happy to update the misspellings. If you are looking for someone to do things like update every instance of a filename and add or remove the full path, I'm not the right person for that type of editing.

@jxmx
Copy link
Member

jxmx commented Dec 28, 2024

We should always be using full paths to files. They should be updated/fixed by the requestor at merge time.

@knobby4444
Copy link
Collaborator Author

It feels to me that with the amount of documentation updates that are needed just to get to a point where people can be successful by using the documentation, perhaps a new wiki or something that's easier to update would better serve the project.

@knobby4444 knobby4444 closed this Jan 2, 2025
@jxmx
Copy link
Member

jxmx commented Jan 2, 2025

@knobby4444 The great thing about documentation platforms is no one reads it regardless of how you publish it...

@jxmx jxmx reopened this Jan 2, 2025
@Allan-N Allan-N merged commit ca841fc into main Jan 11, 2025
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.

3 participants