Commit Graph
3 Commits
Author SHA1 Message Date
Claude cb95676722 Fix kiosk audio output: don't mute microphone by default
The PTT handler was muting the audio source before checking if a PTT
device was configured. This prevented kiosks from sending audio back
to calling phones during normal intercom operation.

Changes:
- Move audio mute command after PTT device check
- Only mute audio when PTT device is actually configured
- Leave audio unmuted for standard kiosk intercom functionality
- Add clarifying comments about PTT mode vs normal operation

This fixes the issue where phones calling the kiosk could send audio
but received no audio back from the kiosk.
2025-12-03 17:14:22 +00:00
Claude 74f9f63828 CRITICAL FIX: Set ENABLE_TLS="n" by default for LAN-only installs
ROOT CAUSE FOUND:
- install_server_only() was hardcoding ENABLE_TLS="y" (line 2039)
- install_full() wasn't setting ENABLE_TLS at all
- This caused devices to always use TLS/5061 even for LAN-only setups

FIXES:
1. install_server_only(): Changed ENABLE_TLS="y" to ENABLE_TLS="n"
2. install_full(): Added ENABLE_TLS="n" before configure_asterisk
3. Added debug output showing ENABLE_TLS and DOMAIN_NAME values in device summary

HOW IT WORKS NOW:
- Both install functions start with ENABLE_TLS="n" (LAN-only/UDP)
- Only setup_internet_access() sets ENABLE_TLS="y" (internet/TLS)
- Device configuration respects ENABLE_TLS value from saved config
- Device summary correctly displays UDP/5060 for LAN-only
- Device summary shows TLS/5061 only after internet/certs setup

This ensures devices use UDP transport for LAN-only installations
and TLS only when explicitly configured for internet calling.
2025-12-03 16:30:58 +00:00
Claude cd18ca5e5d 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.
2025-12-03 16:15:38 +00:00