Added the code to handle panic while destroying the pubnub client in case pn.jobQueue is already closed

piyushkumar96 commented 4 months ago

Type of Change

Fixture to handle PubNub Client Destroy Gracefully in case JobQueue is already closed(non-breaking changes which adds functionality)

Description

Problem

I was implementing the reinitialize pubnub client mechanism in my application, So before reinitializing it, I was calling the inbuilt Destroy() func for cleaning all open requests, removing listeners, etc. It was throwing panic with the error mentioned below
Code In which it was panicking p.client.Destroy() lib code snapshot panic Error

Solution

Added the condition to check if it is already closed or not before closing it

Tested

Tested locally I was able to destroy the client connection gracefully and reinitialize the client connection in case of any pubnub system status events

piyushkumar96 commented 1 month ago

Hi @marcin-cebo @Xavrax @seba-aln Can anyone one of you please review this PR, Please let me know if you need anything else from me, feel free to reach out to me. Thanks & Regards, Piyush

piyushkumar96 commented 2 weeks ago

Hi @marcin-cebo @Xavrax @seba-aln Still waiting to get this PR review by you, Could you guys please review it. Please let me know if you need anything else from me, feel free to reach out to me. Thanks & Regards, Piyush

pubnub / go