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

$cfg['Servers'][$i]['SignonScript'] is broken #13400

Closed
lbausch opened this issue Jun 21, 2017 · 4 comments
Closed

$cfg['Servers'][$i]['SignonScript'] is broken #13400

lbausch opened this issue Jun 21, 2017 · 4 comments
Assignees
Milestone

Comments

@lbausch
Copy link

lbausch commented Jun 21, 2017

Steps to reproduce

  1. Set $cfg['Servers'][$i]['auth_type'] = 'signon';
  2. Set $cfg['Servers'][$i]['SignonScript'] = './myscript.php';
  3. Open phpMyAdmin in browser

Expected behaviour

myscript.php should be sourced and used to obtain login credentials

Actual behaviour

Error page is displayed, saying

phpMyAdmin - Error
You must set SignonURL!

Server configuration

Operating system: Debian 7.11

Web server: Apache 2.2.22

Database: MySQL 5.5.55

PHP version: PHP 5.4

phpMyAdmin version: 3.4.11.1deb2+deb7u8

Client configuration

Browser: Chromium

Operating system: Debian

@nijel
Copy link
Contributor

nijel commented Jun 28, 2017

Apparently you don't have set $cfg['Servers'][$i]['SignonURL'], please set it.

You get to this point because SignonScript didn't return valid credentials and phpMyAdmin tries to redirect to SignonURL.

@nijel nijel closed this as completed Jun 28, 2017
@nijel nijel self-assigned this Jun 28, 2017
@nijel nijel added the question Used when we need feedback from the submitter or when the issue is a question about PMA label Jun 28, 2017
@lbausch
Copy link
Author

lbausch commented Jun 28, 2017

The SignonScript is never sourced - that's the problem.

@lbausch
Copy link
Author

lbausch commented Jun 28, 2017

if SignonURL is set an instant redirect happens - and SignonScript is ignored.

Am I misreading the docs here?

@nijel
Copy link
Contributor

nijel commented Jun 28, 2017

Oh, I've just noticed that you're using quite old version, that doesn't support SignonScript, it was introduced later in 72886c3.

nijel added a commit that referenced this issue Jun 28, 2017
See #13400

Signed-off-by: Michal Čihař <michal@cihar.com>
@nijel nijel added documentation and removed question Used when we need feedback from the submitter or when the issue is a question about PMA labels Jun 28, 2017
@nijel nijel added this to the 4.7.2 milestone Jun 28, 2017
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants