Project

General

Profile

Actions

Bug #51540

closed

Check for host before connect to database

Added by Nicole Cordes over 10 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
Install Tool
Target version:
Start date:
2013-08-29
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

On Windows system a database connection with an empty host is possible and a link is returned. We have to add a check for empty hostname otherwise an installation of TYPO3 isn't possible.


Related issues 3 (0 open3 closed)

Related to TYPO3 Core - Task #49162: Rewrite install toolClosed2011-10-30

Actions
Related to TYPO3 Core - Bug #51956: Install Tool does not ask for database settingsClosed2013-09-12

Actions
Is duplicate of TYPO3 Core - Bug #51433: Install tool: Refactor database connect configuration settingsClosed2013-08-28

Actions
Actions #1

Updated by Nicole Cordes over 10 years ago

An empty string is taken as 'localhost'.

Actions #2

Updated by Christian Kuhn over 10 years ago

This is probably solved with the pending patch for #51433

Actions #3

Updated by Christian Kuhn over 10 years ago

  • Status changed from New to Closed

closed as dupe for now.

Actions #4

Updated by Gerrit Code Review over 10 years ago

  • Status changed from Closed to Under Review

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

Actions #5

Updated by Christian Kuhn over 10 years ago

  • Status changed from Under Review to Closed
  • Is Regression set to No
Actions #6

Updated by Nicole Cordes over 10 years ago

  • Status changed from Closed to Accepted

Tested current master with a standard xampp on Windows. Still same behaviour. But instead of checking for an empty host IMHO it's better to tune the install wizard

Actions #7

Updated by Gerrit Code Review over 10 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/23866

Actions #8

Updated by Gerrit Code Review over 10 years ago

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

Actions #9

Updated by Gerrit Code Review over 10 years ago

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

Actions #10

Updated by Nicole Cordes over 10 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #11

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF