search

noahheck / E_PDOStatement

Drop in replacement for default PHP PDOStatement class allowing devs to view an interpolated version of a parameterized query
Other
51 stars 13 forks source link

Request aborted... #12

Closed Antjac closed 7 years ago

Antjac commented 8 years ago

Hi,

I've an issue with the last release. When I try to execute it, my page crash (aborted on firebug). It seems to be on the regex : $testParam = "/({$marker}(?!\w))(?=(?:[^\"']|[\"'][^\"'][\"'])$)/";

When i replaced it by the old one ($testParam = "/" . $marker . "(?!\w)/";), it's ok

Apache/2.4.18 (Win32) PHP/5.6.19

noahheck commented 8 years ago

Do you have any error information (stack trace, etc.)? You can output errors by enabling error_reporting and error display:

error_reporting(E_ALL);
ini_set("display_errors", 1);
Antjac commented 8 years ago

No error, Apache just reset...

[Wed Nov 16 16:59:58.019661 2016] [mpm_winnt:notice] [pid 9828:tid 648] AH00428: Parent: child process 15080 exited with status 255 -- Restarting. [Wed Nov 16 16:59:58.111510 2016] [auth_digest:notice] [pid 9828:tid 648] AH01757: generating secret for digest authentication ... [Wed Nov 16 16:59:58.137264 2016] [mpm_winnt:notice] [pid 9828:tid 648] AH00455: Apache/2.4.18 (Win32) PHP/5.6.19 configured -- resuming normal operations [Wed Nov 16 16:59:58.137264 2016] [mpm_winnt:notice] [pid 9828:tid 648] AH00456: Apache Lounge VC14 Server built: Dec 9 2015 10:17:39 [Wed Nov 16 16:59:58.137264 2016] [core:notice] [pid 9828:tid 648] AH00094: Command line: 'c:\wamp\bin\apache\apache2.4.18\bin\httpd.exe -d C:/wamp/bin/apache/apache2.4.18' [Wed Nov 16 16:59:58.140266 2016] [mpm_winnt:notice] [pid 9828:tid 648] AH00418: Parent: Created child process 13708 [Wed Nov 16 16:59:58.602878 2016] [auth_digest:notice] [pid 13708:tid 80] AH01757: generating secret for digest authentication ... [Wed Nov 16 16:59:58.638892 2016] [mpm_winnt:notice] [pid 13708:tid 80] AH00354: Child: Starting 64 worker threads. [Wed Nov 16 16:59:58.903568 2016] [mpm_winnt:notice] [pid 9828:tid 648] AH00428: Parent: child process 13708 exited with status 255 -- Restarting. [Wed Nov 16 16:59:58.953014 2016] [auth_digest:notice] [pid 9828:tid 648] AH01757: generating secret for digest authentication ... [Wed Nov 16 16:59:58.977031 2016] [mpm_winnt:notice] [pid 9828:tid 648] AH00455: Apache/2.4.18 (Win32) PHP/5.6.19 configured -- resuming normal operations [Wed Nov 16 16:59:58.977031 2016] [mpm_winnt:notice] [pid 9828:tid 648] AH00456: Apache Lounge VC14 Server built: Dec 9 2015 10:17:39 [Wed Nov 16 16:59:58.977031 2016] [core:notice] [pid 9828:tid 648] AH00094: Command line: 'c:\wamp\bin\apache\apache2.4.18\bin\httpd.exe -d C:/wamp/bin/apache/apache2.4.18' [Wed Nov 16 16:59:58.980032 2016] [mpm_winnt:notice] [pid 9828:tid 648] AH00418: Parent: Created child process 892 [Wed Nov 16 16:59:59.427802 2016] [auth_digest:notice] [pid 892:tid 504] AH01757: generating secret for digest authentication ... [Wed Nov 16 16:59:59.456327 2016] [mpm_winnt:notice] [pid 892:tid 504] AH00354: Child: Starting 64 worker threads.

noahheck commented 8 years ago

I don't see anything here to help identify any issues with PHP. Can you look in your php error log? I believe WAMP has a shortcut option for viewing the error log. Alternatively, can you wrap your code in a try..catch and attempt to catch any php exceptions?

FYI, the updated regex is necessary to prevent incorrect interpolation for strings that might contain the placeholder string, so if you're using this for production logging, etc, you'll definitely want to be able to use the updated version.

Antjac commented 8 years ago

Nothing in PHP Error... I've just this on the event viewer of Windows 10 (Report is in french)

Nom de l’application défaillante httpd.exe, version : 2.4.18.0, horodatage : 0x5667f02e Nom du module défaillant : php5ts.dll, version : 5.6.19.0, horodatage : 0x56d7bf43 Code d’exception : 0xc00000fd Décalage d’erreur : 0x0011e0e2 ID du processus défaillant : 0x335c Heure de début de l’application défaillante : 0x01d240a68f8310b8 Chemin d’accès de l’application défaillante : C:\wamp\bin\apache\apache2.4.18\bin\httpd.exe Chemin d’accès du module défaillant: C:\wamp\bin\apache\apache2.4.18\bin\php5ts.dll ID de rapport : 8c5f5caf-cd3a-4152-b7cb-f0703571c02a Nom complet du package défaillant : ID de l’application relative au package défaillant :

noahheck commented 7 years ago

This seems like it might be an issue with your instance of WAMP (or perhaps with WAMP itself). Have you tried/are you able to run the test suite for E_PDOStatement? If so, do all tests pass, or is there any helpful output from running the tests?

Copy the tests/config/config.dist.php to tests/config/config.php and edit that file to contain appropriate values for your environment (PDO dsn, username, password, etc).

Make sure all of the composer dependencies are installed for the project:

> /path/to/composer install

then run the test suite from the tests directory:

> /path/to/php ./vendor/bin/phpunit

OR

use the test helper script:

> /path/to/php path/to/tests/run_tests
Antjac commented 7 years ago

I tried to launch the tests but the installation of composer with your json failed because ext-pcntl is missing (t depends of php-invoker). It seems to be a Unix only extension... So impossible to run it on Windows.

noahheck commented 7 years ago

I'm out of my normal here (I work on Unix only), but according to this SO answer (http://stackoverflow.com/questions/30045085/how-to-install-pcntl-extension-in-windows) you may be able to install PHPUnit without installing php-invoker, which might let you run the tests.

Antjac commented 7 years ago

Fatal error: Cannot redeclare class BankAccount in C:\wamp\www\test3\tests\vendor\bin\vendor\phpunit\php-code-coverage\tests_files\BankAccount.php on line 3

Call Stack: 0.0003 125312 1. {main}() C:\wamp\www\test3\vendor\phpunit\phpunit\phpunit:0 0.0043 432304 2. PHPUnit_TextUI_Command::main() C:\wamp\www\test3\vendor\phpunit\phpunit\phpunit:47 0.0043 436360 3. PHPUnit_TextUI_Command->run() C:\wamp\www\test3\vendor\phpunit\phpunit\src\TextUI\Command.php:100 1.0876 2775640 4. PHPUnit_Runner_BaseTestRunner->getTest() C:\wamp\www\test3\vendor\phpunit\phpunit\src\TextUI\Command.php:123 5.1207 3038184 5. PHPUnit_Framework_TestSuite->addTestFiles() C:\wamp\www\test3\vendor\phpunit\phpunit\src\Runner\BaseTestRunner.php:58 5.5500 9756944 6. PHPUnit_Framework_TestSuite->addTestFile() C:\wamp\www\test3\vendor\phpunit\phpunit\src\Framework\TestSuite.php:409 5.5504 9812688 7. PHPUnit_Util_Fileloader::checkAndLoad() C:\wamp\www\test3\vendor\phpunit\phpunit\src\Framework\TestSuite.php:335 5.5513 9812832 8. PHPUnit_Util_Fileloader::load() C:\wamp\www\test3\vendor\phpunit\phpunit\src\Util\Fileloader.php:38 5.5522 9869840 9. include_once('C:\wamp\www\test3\tests\vendor\bin\vendor\phpunit\php-code-coverage\tests\PHP\CodeCoverageTest.php') C:\wamp\www\test3\vendor\phpunit\phpunit\src\Util\Fileloader.php:56 5.5539 9978064 10. require_once('C:\wamp\www\test3\tests\vendor\bin\vendor\phpunit\php-code-coverage\tests_files\BankAccount.php') C:\wamp\www\test3\tests\vendor\bin\vendor\phpunit\php-code-coverage\tests\PHP\CodeCoverageTest.php:20

is it normal ?

noahheck commented 7 years ago

Not really. It looks like you're importing the class definition more than once during the execution of your tests. You should only include that file one time, or implement an autoloader and not have to include the file at all.

Antjac commented 7 years ago

I've a lot of "redeclare" error... So I prefer ditched...