Static singletons used everywhere

[stevebauman]

I'm working on an application that will facilitate reporting against multiple Meta integrations during the same request, but I quickly discovered that this is will cause issues if you keep one integration in memory and call it later after another one has been initialized. For example:

use FacebookAds\Api;

$one = Api::init(...);

$two = Api::init(...); // Overwrites the first

$one->... // API calls will now actually occur on $two

I thought there would be a way around this by setting the HTTP client inside of the EventRequest manually:

facebook-php-business-sdk/src/FacebookAds/Object/ServerSide/EventRequest.php

Lines 214 to 217 in f3eb099

	public function setHttpClient($http_client) {
	$this->http_client = $http_client;
	return $this;
	}

But then the singletons are called regardless later on:

facebook-php-business-sdk/src/FacebookAds/Object/ServerSide/EventRequest.php

Lines 309 to 313 in f3eb099

	if (HttpServiceClientConfig::getInstance()->getAccessToken() == null) {
	$params['access_token'] = Api::instance()->getSession()->getAccessToken();
	} else {
	$params['access_token'] = HttpServiceClientConfig::getInstance()->getAccessToken();
	}

facebook-php-business-sdk/src/FacebookAds/Object/ServerSide/EventRequest.php

Lines 316 to 320 in f3eb099

	if (HttpServiceClientConfig::getInstance()->getAppsecret() == null) {
	$appsecret = Api::instance()->getSession()->getAppSecret();
	} else {
	$appsecret = HttpServiceClientConfig::getInstance()->getAppsecret();
	}

Why is this the case? Why do all these static PHP singletons need to exist? This will surely cause issues for anyone running a PHP application from memory (Swoole, FrankenPHP, RoadRunner, etc.).

[norberttech]

I'm facing similar problem...

@stevebauman did you managed to find a solution for that?

[stevebauman]

@norberttech No I haven't unfortunately.

I'm considering just using raw HTTP calls to their API instead. Doesn't seem like there's an easy way with this SDK without extending a bunch of classes and overriding a ton of methods.

[norberttech]

yeah, I was thinking the same but it also seems that joelbutcher/facebook-graph-sdk works fine, I just tested fetching campaigns and it works, plus it's not a singleton so multitenancy is doable.

$fb = new \Facebook\Facebook([
  'app_id' => $authConfig['app_id'],
  'app_secret' => $authConfig['app_secret'],
  'default_graph_version' => $authConfig['api_version'],
  'default_access_token' => 'long_living_access_token',
  'http_client_handler' => new GuzzleHttpClient(...), // you can also inject your custom client in case you would like to mock responses
]);
            
$accountsData = $facebook->get('/me/adaccounts')->getDecodedBody();
$campaignsData = $facebook->get($accountsData['data'][0]['id'] . '/campaigns')->getDecodedBody();

I'm considering just using raw HTTP calls to their API instead.

Yeah, in case of any issues with this sdk I'm probably also gonna look into simple http request/response that would return associative arrays.