Revert session_serializer_name(), session_gc()

From: Date: Tue, 11 Mar 2014 13:37:44 +0000
Subject: Revert session_serializer_name(), session_gc()
Groups: php.internals 
Request: Send a blank email to [email protected] to get a copy of this message
Hello,

I just saw this in the 5.6-alpha1 changelog:

   - Implemented Request #54649 (Create session_serializer_name()). (Yasuo)
    ...
   - Implemented Request #11100 (session_gc() function). (Yasuo)


I didn't find an RFC and I don't know a way to search for a discussion
on internals (Google doesn't find one).

session_serializer_name(): https://bugs.php.net/bug.php?id=54649

I don't think that this should've been implemented, especially with
the reasoning from the feature request ... The same could be applied
to all session ini options and especially now since session_start()
accepts them, it's useless IMO.

session_gc() was added in a similar fashion:
https://bugs.php.net/bug.php?id=11100

There's already a comment about how the same thing can be achieved by
overloading SessionHandler::gc(), but a user could also just alter
session.gc_divisor, session.gc_probability to ensure that the garbage
collector is started.
And while session_serializer_name() is just redundant, session_gc()
could cause performance issues.

I'd strongly suggest that these 2 functions should be removed before
PHP 5.6 is officially released.

Regards,
Andrey Andreev.


Thread (41 messages)

« previous php.internals (#73066) next »