v1.28: Fix LAN-only device configuration and add informative messages
CRITICAL FIX: - Added load_config() call in add_device_menu to read saved ENABLE_TLS setting - Device display now correctly shows UDP/5060 when ENABLE_TLS != "y" - Device display shows TLS/5061 only when ENABLE_TLS == "y" - SRTP requirement properly displayed based on TLS configuration OTHER FIXES: - ICE support only enabled when DOMAIN_NAME is set (FQDN/internet calling) - RTP config: icesupport and STUN only for FQDN setups, not LAN-only - Added informative message after server install before internet setup prompt ISSUE RESOLVED: When installing server-only without selecting internet/certs, devices now correctly show: Transport: UDP (port 5060) SRTP: Not required This matches the actual pjsip.conf configuration and allows mobile devices to properly register with UDP transport on LAN networks. Created v1.28.sh with all fixes, keeping v1.23.sh for reference.
This commit is contained in:
@@ -559,6 +559,7 @@ manage_rooms() {
|
||||
|
||||
add_device_menu() {
|
||||
print_header "Add Device"
|
||||
load_config # Load saved configuration to check ENABLE_TLS, DOMAIN_NAME, etc.
|
||||
list_categories
|
||||
read -p "Category number: " cat_num
|
||||
local cat_data=$(get_category_by_index "$cat_num")
|
||||
@@ -2005,11 +2006,18 @@ install_full() {
|
||||
enable_client_services
|
||||
open_firewall_ports
|
||||
save_config
|
||||
|
||||
|
||||
echo ""
|
||||
echo "════════════════════════════════════════════════════════"
|
||||
print_success "Local network install complete"
|
||||
echo ""
|
||||
echo "Server and devices are reachable over internal LAN network only."
|
||||
echo "To add internet calling capability, continue with the setup below."
|
||||
echo "════════════════════════════════════════════════════════"
|
||||
echo ""
|
||||
read -p "Run Internet/Certificate Setup wizard now? [Y/n]: " run_setup
|
||||
[[ ! "$run_setup" =~ ^[Nn]$ ]] && setup_internet_access
|
||||
|
||||
|
||||
print_success "Installation complete"
|
||||
}
|
||||
|
||||
@@ -2022,11 +2030,18 @@ install_server_only() {
|
||||
open_firewall_ports
|
||||
INSTALLED_SERVER="y"
|
||||
save_config
|
||||
|
||||
|
||||
echo ""
|
||||
echo "════════════════════════════════════════════════════════"
|
||||
print_success "Local network install complete"
|
||||
echo ""
|
||||
echo "Server and devices are reachable over internal LAN network only."
|
||||
echo "To add internet calling capability, continue with the setup below."
|
||||
echo "════════════════════════════════════════════════════════"
|
||||
echo ""
|
||||
read -p "Run Internet/Certificate Setup wizard now? [Y/n]: " run_setup
|
||||
[[ ! "$run_setup" =~ ^[Nn]$ ]] && setup_internet_access
|
||||
|
||||
|
||||
print_success "Server installed"
|
||||
}
|
||||
|
||||
|
||||
File diff suppressed because it is too large
Load Diff
Reference in New Issue
Block a user