Bug: Supeglobals does not fully support actions

[neznaika0]

PHP Version

8.3

CodeIgniter4 Version

4.6

CodeIgniter4 Installation Method

Git

Which operating systems have you tested for this bug?

Linux

Which server did you use?

cli-server (PHP built-in webserver)

What happened?

At the moment, Superglobals looks like a stub. There are no necessary actions:

1. There is no getting all the values as an array. For example, to replace it in the Router: $this->globals['server'] = $_SERVER;
   

   CodeIgniter4/system/Commands/Utilities/Routes.php

   Lines 89 to 94 in e475fd8

   	if ($host !== null) {
   	$request = service('request');
   	$_SERVER = $request->getServer();
   	$_SERVER['HTTP_HOST'] = $host;
   	$request->setGlobal('server', $_SERVER);
   	}

2. The server() method can return not only a string, but also string|int|float|array. Array for argv, float for REQUEST_TIME
   

   CodeIgniter4/system/Superglobals.php

   Line 33 in e475fd8

   public function server(string $key): ?string

3. Keys cannot be deleted as unset($_SERVER['HTTP_HOST'], $_SERVER['REQUEST_URI'])
   

   CodeIgniter4/system/Commands/Utilities/Environment.php

   Line 121 in e475fd8

   unset($_ENV['CI_ENVIRONMENT'], $_SERVER['CI_ENVIRONMENT']);

4. Something important is in the process..

To upgrade to Superglobals, you need to check the all code. Because in many places, $_SERVER is overwritten. I'll try to get started.

I could send a PR, but I need to understand what to do. Is the function that I described enough for this?

I'm really sorry, this is more of a feature request. But I have problems accessing the forum.

[neznaika0]

@paulbalandan @samsonasik Has a policeman appeared? 😄

[samsonasik]

@paulbalandan it seems due to mergeable PR:

• chore: clean up issue templates #9355

@neznaika0 let's reopen

[michalsn]

From what I see, the Superglobals class was created with the intention to handle all superglobals, so I guess the idea of using it is fine. However, currently, it's used only with the SiteURIFactory class and is handling only these superglobals used for URL manipulation.

We can ask @kenjis if there was a reason we didn't pursue this further. He is definitely the best-oriented person in this area (as the author) so his opinion can save us some time.

[michalsn]

@neznaika0 What exactly are the problems with accessing the forum?

[neznaika0]

@kenjis has been very busy for a long time. We could first describe the interface and then implement the methods. It seems to me that it would be possible to extend ArrayObject or Iterable... The task is to hide the array in an object

I opened a topic in the Lounge on the forum. Cloudflare Encrypted Client Hello (ECH) was banned in my country. I'm 50/50% open to a forum or a long wait. Sometimes it opens normally