session use redis Adapter can't to destroy the data in php7

[bain2018]

when i use the redis as the session Adapter;
i can't destroy all of the session data
phalcon 3.0.1
php 7

[bain2018]

did u finish this bug?

[Yurist-85]

This is a BUG.
Faced with such problem on Phalcon v2 + PHP 5.6.
Look in source code. Incorrect behaviour of this Adapter is from the beginning.
Just compare original Adapter and Redis Adapter.
No need in any row of script "to reproduce" it. It simply works wrong.

[Jurigag]

What you mean no script to reproduce ? Then how to recreate this issue without any code ? For example someone is not using redis adapter - then he doesnt have such a bug. Script to reproduce is surely needed.

[Yurist-85]

There's no need to recreate it. Look in source code. If your brain works fine I'll see that there're absolutely different behaviours of ->destroy() methods in original Adapter and Redis Adapter.

[sergeyklay]

I'm closing this issue due to the lack of any reaction. I'll open it again if the need arises

[gamalan]

I also happen to facing this, i think this is because the implementation of destroy between Adapter and RedisAdapter when destroying is totally different.

in adapter.zep

public function destroy(boolean removeData = false) -> boolean

but in redis.zep

public function destroy(string sessionId = null) -> boolean

and yet in the php stub,

public function destroy($removeData = false)

Can you tell what kind of script you need to reproduce? I'd like to help to help if possible.

The only way to destroy the session is by remove one by one, or using session_destroy. Which is problematic i think.

[gamalan]

also, in both adapter.zep and libmemcached.zep, this is implemented in destroy function.

for key, _ in _SESSION {
	unset _SESSION[key];
}

yet in memcache.zep and redis.zep, this is not implemented.

[sergeyklay]

@gamalan I'll try to sort out asap

[sergeyklay]

@gamalan #12833

[sergeyklay]

Fixed in the 3.2.x branch. Feel free to open new issue if the problem appears again. Thank you for contributing.