Debian Bug report logs - #837897
php5 - sessionclean doesn't take conf.d into account (patch included)

Toggle useless messages

Message #5 received at submit@bugs.debian.org (full text, mbox, reply):

From: Benjamin DUPUIS <poil@quake.fr>

To: submit@bugs.debian.org

Subject: php5 - sessionclean doesn't take conf.d into account (patch included)

Date: Thu, 15 Sep 2016 11:46:43 +0200

Package: php5

Version: 5.6.20+dfsg-0+deb8u1

/usr/lib/php5/sessionclean doesn't take /etc/php5/*/conf.d into account.

So when you set a different value for the session.gc_maxlifetime in a 
conf.d/ in file it's not taken into account and the session are purged.

There is an easily quickfix just set export PHP_INI_SCAN_DIR before 
executing "php5"

--- /usr/lib/php5/sessionclean.orig    2016-09-15 11:43:45.752148700 +0200
+++ /usr/lib/php5/sessionclean    2016-09-15 11:44:48.703538472 +0200
@@ -8,6 +8,7 @@
 printf "$SAPIS" | \
 while IFS=: read -r conf_dir proc_name; do
     if [ -e /etc/php5/${conf_dir}/php.ini ]; then
+        export PHP_INI_SCAN_DIR=/etc/php5/${conf_dir}/conf.d
         # Get all session variables once so we don't need to start PHP 
to get each config option
         session_config=$(php5 -c /etc/php5/${conf_dir}/php.ini -d 
"error_reporting='~E_ALL'" -r 'foreach(ini_get_all("session") as $k => 
$v) echo "$k=".$v["local_value"]."\n";')
         save_handler=$(echo "$session_config" | sed -ne 
's/^session\.save_handler=\(.*\)$/\1/p')

Best regards,

Message #10 received at 837897@bugs.debian.org (full text, mbox, reply):

From: Ondřej Surý <ondrej@sury.org>

To: Benjamin DUPUIS <poil@quake.fr>, 837897@bugs.debian.org

Subject: Re: [php-maint] Bug#837897: php5 - sessionclean doesn't take conf.d into account (patch included)

Date: Thu, 15 Sep 2016 12:31:25 +0200

Control: reassign -1 php5-common
Control: fixed -1 php5/5.6.23+dfsg-0+deb8u1
Control: forcemerge 827548 -1

This has been already fixed in php5/5.6.23+dfsg-0+deb8u1

-- 
Ondřej Surý <ondrej@sury.org>
Knot DNS (https://www.knot-dns.cz/) – a high-performance DNS server
Knot Resolver (https://www.knot-resolver.cz/) – secure, privacy-aware,
fast DNS(SEC) resolver
Vše pro chleba (https://vseprochleba.cz) – Mouky ze mlýna a potřeby pro
pečení chleba všeho druhu

On Thu, Sep 15, 2016, at 11:46, Benjamin DUPUIS wrote:
> Package: php5
> 
> Version: 5.6.20+dfsg-0+deb8u1
> 
> /usr/lib/php5/sessionclean doesn't take /etc/php5/*/conf.d into account.
> 
> So when you set a different value for the session.gc_maxlifetime in a 
> conf.d/ in file it's not taken into account and the session are purged.
> 
> There is an easily quickfix just set export PHP_INI_SCAN_DIR before 
> executing "php5"
> 
> --- /usr/lib/php5/sessionclean.orig    2016-09-15 11:43:45.752148700
> +0200
> +++ /usr/lib/php5/sessionclean    2016-09-15 11:44:48.703538472 +0200
> @@ -8,6 +8,7 @@
>   printf "$SAPIS" | \
>   while IFS=: read -r conf_dir proc_name; do
>       if [ -e /etc/php5/${conf_dir}/php.ini ]; then
> +        export PHP_INI_SCAN_DIR=/etc/php5/${conf_dir}/conf.d
>           # Get all session variables once so we don't need to start PHP 
> to get each config option
>           session_config=$(php5 -c /etc/php5/${conf_dir}/php.ini -d 
> "error_reporting='~E_ALL'" -r 'foreach(ini_get_all("session") as $k => 
> $v) echo "$k=".$v["local_value"]."\n";')
>           save_handler=$(echo "$session_config" | sed -ne 
> 's/^session\.save_handler=\(.*\)$/\1/p')
> 
> Best regards,
> 
> _______________________________________________
> pkg-php-maint mailing list
> pkg-php-maint@lists.alioth.debian.org
> http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-php-maint

Send a report that this bug log contains spam.