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

Added IbmDb2 db2_pclose to disconnect #27

Closed
weierophinney opened this issue Dec 31, 2019 · 4 comments
Closed

Added IbmDb2 db2_pclose to disconnect #27

weierophinney opened this issue Dec 31, 2019 · 4 comments

Comments

@weierophinney
Copy link
Member

Updated the disconnect method to choose db2_pclose or db2_close based on the resource's connectionParameter for persistence.

The public function connect() creates either persistent or non persistent db2 connections based on a connection parameter.

This update uses the same logic in the public function disconnect() to close the connection. Previously persistent connections would not have been closed with the db2_close() call, because they need the db2_pclose() call.


Originally posted by @JargonMan at zendframework/zend-db#333

@weierophinney
Copy link
Member Author

Was this supposed to be closed?


Originally posted by @Ocramius at zendframework/zend-db#333 (comment)

@weierophinney
Copy link
Member Author

Sorry, I'm new at GitHub and thought the pull request was rejected and I needed to create a new one.


Originally posted by @JargonMan at zendframework/zend-db#333 (comment)

@weierophinney
Copy link
Member Author

Hah, ok. You can just send more commits to this branch, and that will
update the patch.

On Tue, 14 Aug 2018, 17:47 JargonMan, [email protected] wrote:

Sorry, I'm new at GitHub and thought the pull request was rejected and I
needed to create a new one.


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
zendframework/zend-db#333 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJakN7hVApH9aE6PaCiLIkrff2QN3Idks5uQvEkgaJpZM4V0pXQ
.


Originally posted by @Ocramius at zendframework/zend-db#333 (comment)

@weierophinney
Copy link
Member Author

This package is considered feature-complete, and is now in security-only maintenance mode, following a decision by the Technical Steering Committee.
If you have a security issue, please follow our security reporting guidelines.
If you wish to take on the role of maintainer, please nominate yourself

If you are looking for an actively maintained package alternative, we recommend:

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

1 participant