-
Notifications
You must be signed in to change notification settings - Fork 401
Open
Description
PubNub client has been failing for us since we upgraded to the latest Webpack, which no longer polyfills nodejs methods.
The issue here is that this library uses a non standard global Buffer method. There may be a case where this is failing for others who have upgraded to the latest compilers too.
Some thoughts:
- Convert to using browser standards for buffers.
- If that is not in the timelines, perhaps make the error more graceful, with the library explicitly checking for if
Bufferexists. I wouldn't be surprised if other users of PubNub are not aware their client started failing due to a webpack upgrade.
As an aside, are there docs on where/how to handle errors like this? We weren't aware of the issue for months as nothing obvious was erroring within the addListener({message method, and logVerbosity:true did not surface any errors.
Lefff and karensg
Metadata
Metadata
Assignees
Labels
No labels
