Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PDO object in the PDOSessionHandler not created #374

Closed
skroczek opened this issue Oct 28, 2015 · 1 comment
Closed

PDO object in the PDOSessionHandler not created #374

skroczek opened this issue Oct 28, 2015 · 1 comment
Milestone

Comments

@skroczek
Copy link
Contributor

Playing with the WebsocketAppDemo, I stumbled over an issue with the VirtualSessionStorage with pdo_sqlite as storage.
It turned out that there was no PDO object in the PDOSessionHandler if the database connection is defined as DSN string. To solve the issue you have to call PdoSessionHandler::open, and since this method is defined in the SessionHandlerInterface it should be save to call it every time (In the MemcacheSessionHandler it always returns true, for example.).
To solve this issue, I created the pull request #351 some time ago.

@cboden
Copy link
Member

cboden commented Oct 30, 2015

Thanks @skroczek! I'll have some time in about a week to review that and other open PRs and release v0.3.4. Thanks for your input and my apologies for the delay.

@cboden cboden added this to the 0.3.4 milestone Dec 22, 2015
@cboden cboden closed this as completed Dec 22, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants