php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Request #61763 Improve/add error messages in RINIT
Submitted: 2012-04-18 13:07 UTC Modified: 2012-05-02 15:40 UTC
From: jpauli@php.net Assigned: uw (profile)
Status: Closed Package: mysqlnd_ms (PECL)
PHP Version: 5.3.10 OS: Linux
Private report: No CVE-ID: None
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: jpauli@php.net
New email:
PHP Version: OS:

 

 [2012-04-18 13:07 UTC] jpauli@php.net
Description:
------------
in RINIT, mysqlnd_ms tries to parse the plugin config file.

Starting from 1.1, it uses a JSON config file, but the error message when the 
file is not found still talks about INI file, leading to confusion. The ini param 
still is "mysqlnd_ms.ini_file" , and should be renamed to "mysqlnd_ms.json_file"

Also, it is *very* annoying that if there is a parse error while parsing JSON 
config file, nothing is reported in user land, and the process just keeps on 
failing later on other errors

Test script:
---------------
Give a wrong plugin config file (mysqlnd_ms.ini_file) and see the error message talking about INI file

Give a right plugin config file ((mysqlnd_ms.ini_file), but containing a parse error in the JSON syntax

Expected result:
----------------
Correct error messages

Actual result:
--------------
Incorrect error messages

Patches

mysqlnd_ms-error_message_improvement (last revision 2012-04-18 13:13 UTC by jpauli@php.net)

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2012-04-18 13:13 UTC] jpauli@php.net
The following patch has been added/updated:

Patch Name: mysqlnd_ms-error_message_improvement
Revision:   1334754826
URL:        https://bugs.php.net/patch-display.php?bug=61763&patch=mysqlnd_ms-error_message_improvement&revision=1334754826
 [2012-05-02 15:40 UTC] uw@php.net
Thanks for the patch! The setting has been renamed, http://docs.php.net/manual/en/mysqlnd-ms.configuration.php#ini.mysqlnd-ms.ini-file .
 [2012-05-02 15:40 UTC] uw@php.net
-Status: Open +Status: Closed -Assigned To: +Assigned To: uw
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Sun Dec 22 11:01:30 2024 UTC