Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Undefined array key "hide_connection_errors" Mysqladmin 5.1.2 #17307

Closed
tm8544 opened this issue Jan 22, 2022 · 2 comments
Closed

Undefined array key "hide_connection_errors" Mysqladmin 5.1.2 #17307

tm8544 opened this issue Jan 22, 2022 · 2 comments
Assignees
Labels
Bug A problem or regression with an existing feature
Milestone

Comments

@tm8544
Copy link

tm8544 commented Jan 22, 2022

Describe the bug

Error Undefined array key "hide_connection_errors" even if $cfg["Servers"][$i]["hide_connection_errors"] = true; exists in config.inc.php

Expected behavior

No error

Server configuration

  • Windows 10 x64 [Version 10.0.19044.1466]
  • Apache 2.4.52
  • 10.6.5-MariaDB - mariadb.org binary distribution
  • PHP version: 8.1.2
  • phpMyAdmin version: 5.1.2

Proposed solution: add 'hide_connection_errors' to array in libraries\classes\Config.php
line 1443:
$shared = [
'port',
'socket',
'compress',
'ssl',
'ssl_key',
'ssl_cert',
'ssl_ca',
'ssl_ca_path',
'ssl_ciphers',
'ssl_verify',
'hide_connection_errors',
];

@svivian
Copy link
Contributor

svivian commented Jan 22, 2022

Looks to be fixed by #17304. However I also noticed that parameter has not been added to config.sample.inc.php which would be useful for new installs.

@MauricioFauth MauricioFauth added this to the 4.9.9 milestone Jan 22, 2022
@MauricioFauth MauricioFauth added the Bug A problem or regression with an existing feature label Jan 22, 2022
@Krakenied
Copy link

Krakenied commented Jan 22, 2022

Looks to be fixed by #17304. However I also noticed that parameter has not been added to config.sample.inc.php which would be useful for new installs.

It was not fixed, #17304 makes PMA ignore hide_connection_errors even if it's set (it should be also fixed using solution mentioned by the issue author)

MauricioFauth added a commit that referenced this issue Jan 22, 2022
Fixes #17307
Closes #17304

Signed-off-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
@MauricioFauth MauricioFauth self-assigned this Jan 22, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug A problem or regression with an existing feature
Projects
None yet
Development

No branches or pull requests

4 participants