Closed Bug 575232 Opened 11 years ago Closed 11 years ago

Custom MySQL port support

Categories

(Cloud Services :: Server: Sync, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: znarfor, Assigned: telliott)

Details

Attachments

(3 files, 3 obsolete files)

User-Agent:       Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; fr; rv:1.9.2.4) Gecko/20100611 Firefox/3.6.4
Build Identifier: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; fr; rv:1.9.2.4) Gecko/20100611 Firefox/3.6.4

In weave server backend, there is no support of MySQL port different than the default one (ie 3306)

Reproducible: Always
Attachment #454492 - Attachment is obsolete: true
Attached patch Patch to do configurable ports (obsolete) — Splinter Review
The current proposal will break any install who doesn't update their config file, which isn't desirable for such a common default. This version preserves the correct port for people who don't have an entry in the config file.
Attachment #454493 - Attachment is obsolete: true
Attachment #454641 - Flags: review?
Attachment #454641 - Flags: review? → review?(tziade)
we can also add WEAVE_MYSQL_AUTH_PORT in 1.0/default_constants.php.dist as François did initially, for documentation purpose. So people will know that the port is now configurable, when they set up their Sync box.
Attachment #454641 - Flags: review?(tziade) → review-
Attachment #454641 - Attachment is obsolete: true
Attachment #455189 - Flags: review?
Attachment #455189 - Flags: review? → review?(tarek)
Attachment #455190 - Flags: review?(tarek)
Attachment #455191 - Flags: review?(tarek)
Attachment #455190 - Flags: review?(tarek) → review+
Attachment #455189 - Flags: review?(tarek) → review+
Attachment #455191 - Flags: review?(tarek) → review+
Attachment #455189 - Attachment is patch: true
Attachment #455189 - Attachment mime type: application/octet-stream → text/plain
You need to log in before you can comment on or make changes to this bug.