Bug #48117

Mysql Socket connection fails.

Added by Jaro von Flocken about 8 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
Database API (Doctrine DBAL)
Target version:
-
Start date:
2013-05-12
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.1
PHP Version:
5.3
Tags:
Complexity:
easy
Is Regression:
Sprint Focus:

Description

A connection via socket fails because the socket part is taken as the port wich ist wrong and results in a connection exception.

From the PHP Dokumentation:
The MySQL server parameter: It can also include a port number. e.g. "hostname:port or a path to a local socket e.g." for the localhost.


Related issues

Related to TYPO3 Core - Task #47455: Separate DB TYPO3_CONF_VARS settings host and portClosedAnja Leichsenring2013-04-22

Actions
Related to TYPO3 Core - Bug #49186: Install tool: DB port setting magicClosed2013-06-242013-06-24

Actions
#1

Updated by Wouter Wolters about 8 years ago

Hi Jaro,

We do not use MySQL but MySQLi in TYPO3 6.1. So there needs to be a change to support the Socket parameter.
http://nl3.php.net/manual/en/mysqli.real-connect.php

I will contact the person who implemented the new code in this area.

#2

Updated by Jaro von Flocken about 8 years ago

Hi Wouter,

this would be the cleanest way having an option for port number OR socket. Otherwise the solution would be detecting the type of the content behind the ":" char (string or integer).

Does it help providing you with a patch?

cheers,

Jaro

#3

Updated by Philipp Gampe about 8 years ago

  • Status changed from New to Needs Feedback

@Jaro, it would be really cool if you would provide a patch.

We need a new install tool setting and support for this setting in the connect method.

If you want, you can also push this to our review server:
http://wiki.typo3.org/Contribution_Walkthrough_Tutorials

#4

Updated by Alexander Opitz about 8 years ago

Does #47455 "Separate DB TYPO3_CONF_VARS settings host and port" helps? (This doesn't have a install option yet).

#5

Updated by Philipp Gampe about 8 years ago

  • Status changed from Needs Feedback to Accepted
  • Complexity set to easy

@Alexander, no as the PHP API is not different ... we need a specific Install Tool setting for socket and we need to use this in bootstrap and DatabaseConnection.

#6

Updated by Philipp Gampe about 8 years ago

  • Assignee set to Philipp Gampe
  • % Done changed from 0 to 20
#7

Updated by Gerrit Code Review about 8 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20933

#8

Updated by Gerrit Code Review almost 8 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20933

#9

Updated by Gerrit Code Review almost 8 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20933

#10

Updated by Gerrit Code Review almost 8 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20933

#11

Updated by Gerrit Code Review almost 8 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20933

#12

Updated by Gerrit Code Review almost 8 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20933

#13

Updated by Gerrit Code Review almost 8 years ago

Patch set 7 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20933

#14

Updated by Gerrit Code Review almost 8 years ago

Patch set 1 for branch TYPO3_6-1 has been pushed to the review server.
It is available at https://review.typo3.org/21973

#15

Updated by Gerrit Code Review almost 8 years ago

Patch set 2 for branch TYPO3_6-1 has been pushed to the review server.
It is available at https://review.typo3.org/21973

#16

Updated by Philipp Gampe almost 8 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 20 to 100
#17

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF