DB: when unix_socket is used, do not use host and port by florisluiten · Pull Request #9298 · matomo-org/matomo · GitHub
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.

Sign up for GitHub

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

Jump to bottom

DB: when unix_socket is used, do not use host and port #9298

Merged
merged 1 commit into from
Nov 29, 2015
Merged

DB: when unix_socket is used, do not use host and port #9298

merged 1 commit into from
Nov 29, 2015

Conversation

florisluiten
Copy link
Contributor

Ref #9055

@florisluiten florisluiten mentioned this pull request Nov 27, 2015
Closed
tsteur added a commit that referenced this pull request Nov 29, 2015
@tsteur
Merge pull request #9298 from florisluiten/pdo_mysql_force_unix_socke…
ed7fd15 
…t_if_set

Unset host+port for MySQL PDO if unix_socket is non-empty
@tsteur tsteur merged commit ed7fd15 into matomo-org:master Nov 29, 2015
@tsteur
Copy link
Member

tsteur commented Nov 29, 2015

Thank you for the pull request and for making Piwik better :) Looks good!

@tsteur tsteur added the not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. label Nov 29, 2015
@tsteur tsteur added this to the 2.15.0 milestone Nov 29, 2015
@florisluiten florisluiten deleted the pdo_mysql_force_unix_socket_if_set branch December 3, 2015 12:19
@mattab mattab changed the title Unset host+port for MySQL PDO if unix_socket is non-empty DB: when unix_socket is used, do not use host and port Jan 29, 2016
@mattab mattab added Bug For errors / faults / flaws / inconsistencies etc. and removed not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. labels Jan 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Milestone
2.15.0
Development

Successfully merging this pull request may close these issues.
3 participants
@florisluiten @tsteur @mattab