ceid don't hide - [Solved]

[PiLLaO]

Hi,

I'm just upgrading into phoenix and I find a problem that I can't find the solution, ceid session don't disappear from url, I use my old configuration but something I'm making wrong.

Phoenix v1.1.0.5
php8.5

Code: Select all

Force Cookie Use	False	
Check SSL Session ID	False	
Check User Agent	False	
Check IP Address	False	
Prevent Spider Sessions	True	
Recreate Session	True

Code: Select all

<?php
  error_reporting(E_ALL);

  const HTTP_SERVER = 'https://www.miweb.com';
  const COOKIE_OPTIONS = [
    'lifetime' => 0,
    'domain' => 'www.miweb.com',
    'path' => '/',
    'samesite' => 'Lax',
    'secure' => true,
  ];
  const DIR_WS_CATALOG = '/';

  date_default_timezone_set('Europe/Madrid');

// If you are asked to provide configure.php details,
// please remove the data below before sharing.
  const DIR_FS_CATALOG = '/home/public_html/miweb.com/';

My phoenixcart is in miweb.com folder, not on root

Any help will be appreciated

[PiLLaO]

I make a fresh install and don't have the problem, so.. something that I make it bring me the problem

[PiLLaO]

Find the problem, making the first change of the commit cause conflict with ultimate seo urls

https://github.com/CE-PhoenixCart/Phoen ... d1bdb9be07

includes/system/segments/application/start_session.php [core file]
includes/classes/usu_init.php [addon file]

So I revert changes from

Code: Select all

(! empty(session_id()) ? session_name() . '=' . session_id() : '')

to

Code: Select all

(defined('SID') ? SID : '')

And set php8.3 as install recomended

[burt]

@BrockleyJohn any insights ?

[BrockleyJohn]

@BrockleyJohn any insights ?

I believe the addon needs updating to remove SID references to make it php 8.4 compatible.

Checking / updating it so it works without SID is on my list

[PiLLaO]

Hi,

Making the change here

Code: Select all

includes/system/segments/application/start_session.php [core file]

don't remove ceid from any core page like index.php, shipping.php, privacy.php....

And making the change here

Code: Select all

includes/classes/usu_init.php [addon file]

don't remove ceid fron any friendly url

Best regards