__destruct() access class properties already destroyed

[fiSCIENCES]

Hello Romain,

Luckily, this was working for years.
I changed the if() line a little bit to make it as a patch. But I'm sure you can work it out more robustly.
See below.

Thank you.

    /**
     * @throws FileMakerException
     */
    public function __destruct()
    {
        // Logout dataAPI session if the token cannot be saved to be reused in next query (cli mode or no active session)
        // If a session handler is set, we assume the token has been saved (session may have been closed before destruct was called)
// Patch by MPLT 07-03-2025 because properties are already destroyed!
        if((!session_id() || php_sapi_name() === 'cli') && $this->token && $this->useDataApi && !$this->sessionHandler) {
//        if ($this->token && $this->useDataApi && !$this->sessionHandler && !session_id() || php_sapi_name() === 'cli') {
            $this->dataApiLogout();
        }
    }

[fiSCIENCES]

Hi Romain,

In fact, the problem is more complex.
The token is not well managed and DataAPI never logout.
So we end up with a lot of connection in FMS until we reach error 812-"Exceeded host's capacity" from requests.
I just removed the if() test so that dataApiLogout() is always called. But this is a patch that is time consuming in POST/GET!

Thank you.