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

Fix error on signon login, need create a cookie with sessionID #13630

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@ricardojlrufino

ricardojlrufino commented Sep 1, 2017

No description provided.

Update signon.php
Fix error on signon login, need create an cookie with sessionID
@phpmyadmin-bot

This comment has been minimized.

Show comment
Hide comment
@phpmyadmin-bot

phpmyadmin-bot Sep 1, 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 e136a4f Sep 1, 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.

@ricardojlrufino ricardojlrufino changed the title from Fix error on signon login, need create an cookie with sessionID to Fix error on signon login, need create a cookie with sessionID Sep 1, 2017

@@ -30,6 +30,7 @@
/* Update another field of server configuration */
$_SESSION['PMA_single_signon_cfgupdate'] = array('verbose' => 'Signon test');
$id = session_id();
setcookie($session_name, $id , 0, "/");

This comment has been minimized.

@nijel

nijel Sep 4, 2017

Member

The session_start is supposed to do this...

@nijel

nijel Sep 4, 2017

Member

The session_start is supposed to do this...

This comment has been minimized.

@ricardojlrufino

ricardojlrufino Sep 4, 2017

Strange, it only worked after I did this.
After about 3 hours trying
``
Versão do cliente de banco de dados: libmysql - 5.5.44-MariaDB
Extensão do PHP: mysqli Documentação
Versão do PHP: 5.4.45

@ricardojlrufino

ricardojlrufino Sep 4, 2017

Strange, it only worked after I did this.
After about 3 hours trying
``
Versão do cliente de banco de dados: libmysql - 5.5.44-MariaDB
Extensão do PHP: mysqli Documentação
Versão do PHP: 5.4.45

This comment has been minimized.

@nijel

nijel Sep 4, 2017

Member

What is your setting for session.use_cookies? Does calling following prior to session_start fix your problem?

@ini_set('session.use_cookies', 'true');
@nijel

nijel Sep 4, 2017

Member

What is your setting for session.use_cookies? Does calling following prior to session_start fix your problem?

@ini_set('session.use_cookies', 'true');

@nijel nijel self-assigned this Sep 4, 2017

@nijel

This comment has been minimized.

Show comment
Hide comment
@nijel

nijel Sep 28, 2017

Member

Closing as this should not be really needed. There might be other issue in the script, but this is certainly not the correct solution.

Member

nijel commented Sep 28, 2017

Closing as this should not be really needed. There might be other issue in the script, but this is certainly not the correct solution.

@nijel nijel closed this Sep 28, 2017

nijel added a commit that referenced this pull request Sep 28, 2017

Change example script to use cookies for session
Issue #13630

Signed-off-by: Michal Čihař <michal@cihar.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment