Merge pull request #271 from TECH7Fox/fix-sounds-permissions

Fix downloaded sounds permissions

asterisk/CHANGELOG.md

@@ -22,7 +22,8 @@
 - Ensure Asterisk Mailbox Server is started with `--verbose` if log level is debug or higher
 - Fix Asterisk Mailbox Server media directory to `/media/asterisk/voicemail/default/<mailbox-extension>/` for persistency
 - Fix Asterisk Mailbox Server cache file to `/data/tmp` for persistency
-- Fix to the `/data` directory (and also `/media` when needed) are created on startup, useful when running the add-on as a standalone docker container.
+- Fix to the `/data` directory (and also `/media` when needed) are created on startup, useful when running the add-on as a standalone docker container
+- Fix the permissions of downloaded sounds to be `0755` to be as indicated by <https://asterisksounds.org>, although on my tests it was working before as well.
 
 ### Upgrades
 

asterisk/rootfs/etc/cont-init.d/asterisk.sh

@@ -210,8 +210,6 @@ for sound in "${additional_sounds[@]}"; do
 
     if [[ -f "${lang_file}" && "$(cat "${lang_file}")" == "${sound}" ]]; then
         bashio::log.info "Skipping sounds download for '${sound}'..."
-        rm -rf "${asterisk_sound_dir}"
-        ln -svf "${sound_dir}" "${asterisk_sound_dir}"
     else
         bashio::log.info "Downloading '${sound}' sounds to '${sound_dir}'..."
 
@@ -244,10 +242,10 @@ for sound in "${additional_sounds[@]}"; do
 
         cd - >/dev/null || exit 1
 
+        chmod 0755 -R "${temp_dir}"
         rsync -a "${temp_dir}/" "${sound_dir}/"
-        rm -rf "${asterisk_sound_dir}"
-        ln -svf "${sound_dir}" "${asterisk_sound_dir}"
-
         rm -rf "${temp_dir}"
     fi
+    rm -rf "${asterisk_sound_dir}"
+    ln -svf "${sound_dir}" "${asterisk_sound_dir}"
 done