Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugfix: Socket Consumer Debug: Prevent PHP Warning: Undefined array key 1 #76

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

defunctl
Copy link

This prevents a PHP warning when using the Socket Consumer and have debugging enabled. It seems sometimes the response does not contain an HTTP status and just returns 1 as the body.

Error Logs

appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 94 ] : destruct flush attempt #1
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 113 ] : Flush called - queue size: 2
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 146 ] : Creating new socket at 1705518422
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 163 ] : Opening socket connection to ssl://api.mixpanel.com:443
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 218 ] : Socket wrote 922 bytes
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 135 ] : Batch of 2 consumed, queue size is now 0
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 94 ] : destruct flush attempt #1
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 113 ] : Flush called - queue size: 2
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 146 ] : Creating new socket at 1705518422
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 163 ] : Opening socket connection to ssl://api.mixpanel.com:443
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 218 ] : Socket wrote 1538 bytes
appserver_1  | NOTICE: PHP message: PHP Warning:  Undefined array key 1 in /app/vendor/mixpanel/mixpanel-php/lib/ConsumerStrategies/SocketConsumer.php on line 291

This is often paired with a PHP Warning of PHP message: PHP Warning: fwrite(): SSL: Connection timed out in /app/vendor/mixpanel/mixpanel-php/lib/ConsumerStrategies/SocketConsumer.php on line 215, but not always.

appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 94 ] : destruct flush attempt #1
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 113 ] : Flush called - queue size: 2
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 146 ] : Creating new socket at 1705517840
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 163 ] : Opening socket connection to ssl://api.mixpanel.com:443
appserver_1  | NOTICE: PHP message: PHP Warning:  fwrite(): SSL: Connection timed out in /app/vendor/mixpanel/mixpanel-php/lib/ConsumerStrategies/SocketConsumer.php on line 215
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 218 ] : Socket wrote 0 bytes
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 245 ] : Retrying socket write...
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 146 ] : Creating new socket at 1705517840
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 163 ] : Opening socket connection to ssl://api.mixpanel.com:443
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 218 ] : Socket wrote 922 bytes
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 135 ] : Batch of 2 consumed, queue size is now 0
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 94 ] : destruct flush attempt #1
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 113 ] : Flush called - queue size: 2
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 146 ] : Creating new socket at 1705517840
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 163 ] : Opening socket connection to ssl://api.mixpanel.com:443
appserver_1  | NOTICE: PHP message: [ Base_MixpanelBase - line 218 ] : Socket wrote 1570 bytes
appserver_1  | NOTICE: PHP message: PHP Warning:  Undefined array key 1 in /app/vendor/mixpanel/mixpanel-php/lib/ConsumerStrategies/SocketConsumer.php on line 291

Note: I'm not sure what the best way is to fix the SSL: Connection timed out warning, but I wanted to note it here.

@defunctl defunctl changed the title Socket Consumer Debug: Prevent PHP Warning: Undefined array key 1 Bugfix: Socket Consumer Debug: Prevent PHP Warning: Undefined array key 1 Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant