In my project I have lots of call to SugarCRM via this module.

The fact is we just faced that when SugarCRM is down the site become really slow, this is due to every request waiting for 30 seconds timeout. I tried to lower the value to 5 seconds timeout, noticed some improvements but the site is still slower than when SugarCRM is up.

I am aware that all request doesn't use the session parameter but most of them do. I think we should avoid throwing a request if we have a parameter session set to NULL because this mean we are not able to communicate with SugarCRM yet.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

theo_’s picture

Here is a little patch to avoid throwing a request if the session is null

SocialNicheGuru’s picture

Status: Active » Needs review