fernando/luos
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

BUG 14801 Requerimiento de compatibilidad con Safari de Mac SOLVED

Browse Source 
- La computadora del cliente tiene una hora muy diferente a la hora del servidor, por lo que las cookies creadas son inmediatamente deshechadas por el navegador

- La validcion adicional para las cookies no se aplica cuando se tiene la bandera "safari_cookie_lifetime" con valor "0"

- OJO. ESTE CAMBIO SOLO ARREGLA EL PROBLEMA EXISTENTE EN VIVA PARA LA AUTENTIFICACION DE USUARIOS, CUALQUIER OTRO ISSUE RELACIONADO A FUNCIONAMIENTO DE PM EN SAFARI NO ESTA GARANTIZADO (ESPECIALMENTE ISSUES DE JAVASCRIPT), YA QUE PM NO ES OFICIALMENTE PROBADO EN PM Y NO SABEMOS HASTA CUANDO SEA ASI, ASI QUE FUTUROS ISSUES EN SAFARI SERAN ENVIADOS AL ROADMAP Y DEBE SER APROBADO POR INSTANCIAS SUPERIORES EL VISTO BUENO PARA IR ADELANTE CON LAS PRUEBAS EN SAFARI
This commit is contained in:
Julio Cesar Laura
2014-05-14 13:27:08 -04:00
parent 85570fb3df
commit 6658d57b22
2 changed files with 13 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
workflow/public_html/sysGeneric.php
View File

@@ -285,7 +285,10 @@ if (is_null($timelife)) {
$timelife = 1440;
}
ini_set('session.gc_maxlifetime', $timelife);
if (preg_match("/msie/i", $_SERVER ['HTTP_USER_AGENT']) != 1 || $config['ie_cookie_lifetime'] == 1) {
if ((preg_match("/msie/i", $_SERVER ['HTTP_USER_AGENT']) != 1 ||
$config['ie_cookie_lifetime'] == 1) &&
(!(preg_match("/safari/i", $_SERVER ['HTTP_USER_AGENT']) == 1 && preg_match("/chrome/i", $_SERVER ['HTTP_USER_AGENT']) == 0) ||
$config['safari_cookie_lifetime'] == 1)) {
ini_set('session.cookie_lifetime', $timelife);
}
session_start();
@@ -888,7 +891,10 @@ if (! defined( 'EXECUTE_BY_CRON' )) {
define( 'SYS_LANG_DIRECTION', $oServerConf->getLanDirection() );
if ((isset( $_SESSION['USER_LOGGED'] )) && (! (isset( $_GET['sid'] )))) {
if (preg_match("/msie/i", $_SERVER ['HTTP_USER_AGENT']) != 1 || $config['ie_cookie_lifetime'] == 1) {
if ((preg_match("/msie/i", $_SERVER ['HTTP_USER_AGENT']) != 1 ||
$config['ie_cookie_lifetime'] == 1) &&
(!(preg_match("/safari/i", $_SERVER ['HTTP_USER_AGENT']) == 1 && preg_match("/chrome/i", $_SERVER ['HTTP_USER_AGENT']) == 0) ||
$config['safari_cookie_lifetime'] == 1)) {
if (PHP_VERSION < 5.2) {
setcookie(session_name(), session_id(), time() + $timelife, '/', '; HttpOnly');
} else {
@@ -957,7 +963,10 @@ if (! defined( 'EXECUTE_BY_CRON' )) {
$_SESSION['USER_LOGGED'] = $aUser['USR_UID'];
$_SESSION['USR_USERNAME'] = $aUser['USR_USERNAME'];
$bRedirect = false;
if (preg_match("/msie/i", $_SERVER ['HTTP_USER_AGENT']) != 1 || $config['ie_cookie_lifetime'] == 1) {
if ((preg_match("/msie/i", $_SERVER ['HTTP_USER_AGENT']) != 1 ||
$config['ie_cookie_lifetime'] == 1) &&
(!(preg_match("/safari/i", $_SERVER ['HTTP_USER_AGENT']) == 1 && preg_match("/chrome/i", $_SERVER ['HTTP_USER_AGENT']) == 0) ||
$config['safari_cookie_lifetime'] == 1)) {
if (PHP_VERSION < 5.2) {
setcookie(session_name(), session_id(), time() + $timelife, '/', '; HttpOnly');
} else {