Merge branch '2.3.x' into 2.3.x-saas

CREDITS

@@ -2,7 +2,7 @@
 CREDITS
 =======
 
-Version 2.3.0
+Version 2.3.0/2.3.1
 -------------
 Martin Konecny (martin.konecny@sourcefabric.org)
   Role: Developer Team Lead

changelog

@@ -1,3 +1,16 @@
+2.3.1 - March 19th, 2013
+  * Bug fixes
+    * Security fixes with running unescaped shell commands
+	* Fix backend services not coming back online if RabbitMQ crashed and
+	then restarted
+	* Fix uninstall not properly cleaning system
+	* Improved Services watchdog (test whether Services are running and
+	responding, not just running)
+	* Much faster library import (Silan analyzer runs in background)
+	* Fixed zombie process sometimes being created
+  * Other
+    * Upgrade to Mutagen (tag reader) 1.21 
+
 2.3.0 - Jan 21st, 2013
   * New features
     * Localization (Chinese, Czech, English, French, German, Italian, Korean,

debian/changelog

@@ -1,8 +1,15 @@
+airtime (2.3.0-2) unstable; urgency=low
+
+  * Don't run the airtime-install script if the user has chosen not to
+    set up Apache
+
+ -- Daniel James <daniel@64studio.com>  Tue, 19 Mar 2013 16:39:23 +0000
+
 airtime (2.3.0-1) unstable; urgency=low
 
-  * Nightly development snapshot of Airtime 2.3.x
+  * Upstream 2.3.0 release
 
- -- Daniel James <daniel@64studio.com>  Tue, 22 Jan 2013 11:44:57 +0000
+ -- Daniel James <daniel@64studio.com>  Tue, 12 Feb 2013 11:44:57 +0000
 
 airtime (2.2.1-1) unstable; urgency=low
 

debian/postinst

@@ -1,10 +1,14 @@
 #!/bin/bash
 #postinst script for airtime
 
-set -x
+set -e
 
 . /usr/share/debconf/confmodule
 
+if [ "$DPKG_DEBUG" = "developer" ]; then
+    set -x
+fi
+
 wwwdir="/usr/share/airtime"
 tmpdir="/var/lib/airtime/tmp"
 configdir="/etc/airtime"
@@ -20,39 +24,30 @@ case "$1" in
     webserver="apache2"
     php="php5"
 
-    #  clean up previous configurations
-    if [ -L /etc/$webserver/conf.d/airtime.conf ]; then
-      rm -f /etc/$webserver/conf.d/airtime.conf
-    fi
-
-    if [ -f /etc/$webserver/sites-available/airtime-vhost ]; then
-      a2dissite airtime-vhost
-    fi
-
     # this file in 1.8.2 is a directory path in 1.9.3
     if [ -f /var/www/airtime/utils/airtime-import ]; then
       rm -f /var/www/airtime/utils/airtime-import
     fi
 
-    # APACHE config
+    # do we set up a virtual host?
-    echo "Setting up apache2..."
-
-    # create the document root if it doesn't exist
-    if [ ! -d $wwwdir/public/ ]; then
-      install -d -m755 $wwwdir/public/
-    fi
-
-    # set up the virtual host
     db_get airtime/apache-setup
-    if [ "$RET" == "system-wide (all vhosts)" ]; then
+    APACHESETUP=$RET
-      if [ ! -d /etc/$webserver/conf.d/ ]; then
+    if [ "${APACHESETUP}" == "no thanks" ]; then
-        install -d -m755 /etc/$webserver/conf.d/
+    echo "Not setting up ${webserver} and ${php}..."
-      fi
+
-      if [ ! -e /etc/$webserver/conf.d/airtime.conf ]; then
+     elif [ "${APACHESETUP}" == "dedicated v-host" ]; then
-        ln -s ${includefile} /etc/$webserver/conf.d/airtime.conf
+     echo "Setting up ${webserver}..."
+
+      # create the document root if it doesn't exist
+      if [ ! -d $wwwdir/public/ ]; then
+        install -d -m755 $wwwdir/public/
+      fi
+
+      # temporarily disable an existing virtual host
+      if [ -f /etc/$webserver/sites-available/airtime-vhost ]; then
+        a2dissite airtime-vhost
       fi
 
-    elif [ "$RET" == "dedicated v-host" ]; then
       db_get airtime/apache-servername
       SN=$RET
       db_get airtime/apache-serveradmin
@@ -69,30 +64,49 @@ case "$1" in
       if [ $RETVAL -eq 0 ]; then
       a2ensite airtime-vhost
       fi
-    fi
 
-    # enable the rewrite module
+      # insert a specific hostname, if provided, into API configuration
-    command -v a2enmod > /dev/null
+      if [ "${SN}" != "localhost" ]; then
-    RETVAL=$?
-    if [ $RETVAL -eq 0 ]; then
-      a2enmod rewrite
-    fi
 
-    # remove the default site, if requested to
+        # new installs
-    db_get airtime/apache-deldefault
+        if [ -f /var/lib/airtime/tmp/airtime_mvc/build/airtime.conf -a -f /var/lib/airtime/tmp/python_apps/api_clients/api_client.cfg ]; then
-    if [ "$RET" == "remove default" ]; then
+        sed -i "s/base_url = localhost/base_url = ${SN}/" /var/lib/airtime/tmp/airtime_mvc/build/airtime.conf
-      if [ -f /etc/apache2/sites-available/default ]; then
+        sed -i "s/host = 'localhost'/host = '${SN}'/" /var/lib/airtime/tmp/python_apps/api_clients/api_client.cfg
-      a2dissite default
+        fi
+
+        # upgrades
+        if [ -f /etc/airtime/airtime.conf -a -f /etc/airtime/api_client.cfg ]; then
+        sed -i "s/base_url = localhost/base_url = ${SN}/" /etc/airtime/airtime.conf
+        sed -i "s/host = 'localhost'/host = '${SN}'/" /etc/airtime/api_client.cfg
+        fi
       fi
-    fi
 
-    # PHP config
+      # enable the rewrite module
-    echo "Configuring php5..."
+      command -v a2enmod > /dev/null
-    if [ ! -d /etc/$php/conf.d/ ]; then
+      RETVAL=$?
-      install -d -m755 /etc/$php/conf.d/
+      if [ $RETVAL -eq 0 ]; then
-    fi
+        a2enmod rewrite
-    if [ ! -e /etc/$php/conf.d/airtime.ini ]; then
+      fi
-      ln -s ${phpinifile} /etc/$php/conf.d/airtime.ini
+
+      # remove the default site, if requested to
+      db_get airtime/apache-deldefault
+      if [ "$RET" == "remove default" ]; then
+        if [ -f /etc/apache2/sites-available/default ]; then
+        a2dissite default
+        fi
+      fi
+
+      # PHP config
+      echo "Configuring php5..."
+      if [ ! -d /etc/$php/conf.d/ ]; then
+        install -d -m755 /etc/$php/conf.d/
+      fi
+      if [ ! -e /etc/$php/conf.d/airtime.ini ]; then
+        ln -s ${phpinifile} /etc/$php/conf.d/airtime.ini
+      fi
+
+      # restart apache
+      invoke-rc.d apache2 restart
     fi
 
     # XXX ICECAST XXX
@@ -186,9 +200,6 @@ case "$1" in
      echo "The rabbitmq-server package does not appear to be installed on this server."
     fi
 
-    # restart apache
-    invoke-rc.d apache2 restart
-
     # fix the Liquidsoap symlink if it doesn't point to standard location
     if [ -h /usr/bin/airtime-liquidsoap ]; then
       SYMLINK_TARGET=`readlink /usr/bin/airtime-liquidsoap`
@@ -214,8 +225,12 @@ case "$1" in
       echo "Reconfiguration complete."
     else
 
-      if [ -n "$OLDVERSION" ] && [[ "${OLDVERSION:0:3}" < "1.9" ]]; then
+      if [ -n "$OLDVERSION" ] && [[ "${OLDVERSION:0:3}" < "2.1" ]]; then
-        echo "Upgrades from Airtime versions before 1.9.0 are not supported. Please back up your files and perform a clean install."
+        echo "Upgrades from Airtime versions before 2.1.0 are not supported. Please back up your files and perform a clean install."
+
+      elif [ "${APACHESETUP}" == "no thanks" ]; then
+        echo "Please run the ${tmpdir}/install_minimal/airtime-install script after you have set up the web server."
+
       else
 
         mkdir -p /var/log/airtime

python_apps/pypo/airtime-liquidsoap-init-d

@@ -24,10 +24,12 @@ start () {
 stop () {
         monit unmonitor airtime-liquidsoap >/dev/null 2>&1
         #send term signal after 10 seconds
-        timeout 10 /usr/lib/airtime/airtime_virtualenv/bin/python /usr/lib/airtime/pypo/bin/liquidsoap_scripts/liquidsoap_prepare_terminate.py        
+        timeout -k 5 10 /usr/lib/airtime/airtime_virtualenv/bin/python \
+            /usr/lib/airtime/pypo/bin/liquidsoap_scripts/liquidsoap_prepare_terminate.py        
         # Send TERM after 5 seconds, wait at most 30 seconds.
         start-stop-daemon --stop --oknodo --retry=TERM/10/KILL/5 --quiet --pidfile $PIDFILE
         rm -f $PIDFILE
+        sleep 3
 }
 
 start_no_monit() {

python_apps/pypo/pypofetch.py

@@ -534,7 +534,7 @@ class PypoFetch(Thread):
                 #check if this file is opened (sometimes Liquidsoap is still
                 #playing the file due to our knowledge of the track length
                 #being incorrect!)
-                if not self.is_file_opened():
+                if not self.is_file_opened(path):
                     os.remove(path)
             except Exception, e:
                 self.logger.error(e)

utils/airtime-check-system.php

@@ -99,10 +99,13 @@ class AirtimeCheck {
     {
         $headerInfo = get_headers("http://$p_baseUrl:$p_basePort",1);
         
-        if (!isset($headerInfo['Server'][0]))
+        if (!isset($headerInfo['Server'][0])) {
             return self::UNKNOWN;
-        else
+        } else if (is_array($headerInfo['Server'])) {
             return $headerInfo['Server'][0];
+        } else {
+            return $headerInfo['Server'];
+        }
     }
 
     public static function GetStatus($p_baseUrl, $p_basePort, $p_baseDir, $p_apiKey){