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

config.inc.php add more server,host not current #13760

Closed
wants to merge 1 commit into
base: QA_4_7
from

Conversation

Projects
None yet
3 participants
@y284663247

y284663247 commented Oct 19, 2017

$i++;
/* Authentication type */
$cfg['Servers'][$i]['auth_type'] = 'config';
/* Server parameters */
$cfg['Servers'][$i]['host'] = 'test';
$cfg['Servers'][$i]['compress'] = false;
$cfg['Servers'][$i]['AllowNoPassword'] = false;

/**
 * phpMyAdmin configuration storage settings.
 */

/* User used to manipulate with storage */
 $cfg['Servers'][$i]['controlhost'] = '';
 $cfg['Servers'][$i]['controlport'] = '';
 $cfg['Servers'][$i]['controluser'] = 'pma';
 $cfg['Servers'][$i]['controlpass'] = 'pmapass';

use config.inc.php autologin,

$cfg['Servers'][$i]['controlhost'] 

not used and can't login

edit the code to autologin more database server

Before submitting pull request, please check that every commit:

  • Has proper Signed-Off-By
  • [+] Has commit message which describes it
  • [+] Is needed on it's own, if you have just minor fixes to previous commits, you can squash them
  • [+] Any new functionality is covered by tests
@phpmyadmin-bot

This comment has been minimized.

Show comment
Hide comment
@phpmyadmin-bot

phpmyadmin-bot Oct 19, 2017

This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check contributing documentation for more information.

phpmyadmin-bot commented on 402a9bd Oct 19, 2017

This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check contributing documentation for more information.

@codecov

This comment has been minimized.

Show comment
Hide comment
@codecov

codecov bot Oct 19, 2017

Codecov Report

Merging #13760 into QA_4_7 will decrease coverage by <.01%.
The diff coverage is 60%.

@@            Coverage Diff             @@
##           QA_4_7   #13760      +/-   ##
==========================================
- Coverage   52.71%   52.71%   -0.01%     
==========================================
  Files         467      467              
  Lines       80303    80307       +4     
==========================================
+ Hits        42329    42331       +2     
- Misses      37974    37976       +2

codecov bot commented Oct 19, 2017

Codecov Report

Merging #13760 into QA_4_7 will decrease coverage by <.01%.
The diff coverage is 60%.

@@            Coverage Diff             @@
##           QA_4_7   #13760      +/-   ##
==========================================
- Coverage   52.71%   52.71%   -0.01%     
==========================================
  Files         467      467              
  Lines       80303    80307       +4     
==========================================
+ Hits        42329    42331       +2     
- Misses      37974    37976       +2
@nijel

This comment has been minimized.

Show comment
Hide comment
@nijel

nijel Oct 19, 2017

Member

The control* settings are for different purpose - they should have access only to phpMyAdmin configuration storage. If yo want to use just one setting, stick with password/host, not controlpassword/controlhost, it will work just fine.

Not mentioning that it really doesn't make sense to choose random subset of control settings and apply them to normal connection.

Member

nijel commented Oct 19, 2017

The control* settings are for different purpose - they should have access only to phpMyAdmin configuration storage. If yo want to use just one setting, stick with password/host, not controlpassword/controlhost, it will work just fine.

Not mentioning that it really doesn't make sense to choose random subset of control settings and apply them to normal connection.

@nijel nijel closed this Oct 19, 2017

@nijel nijel self-assigned this Oct 19, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment