BewareMyPower commented on PR #300: URL: https://github.com/apache/pulsar-client-node/pull/300#issuecomment-1441313763
> I don't think we should block the release of 1.8.1 Totally I agree. But the comment [here](https://lists.apache.org/thread/pr9921vflxg96yj0vf870rqptgg05wz1) from @nicoloboschi deserves the discussion. > I wonder why it wasn't needed in 1.7.0 and now it's required in 1.8.1. > This can be problematic for users because the client upgrade requires code > changes and redeployment. The new way of installing the Node.js client brings the requirement of code changes. I think the only way is to copying the cert file to correct paths. Then I think we must document it well that which cert file path should users configure for these systems: - Windows - Linux (including several distributions) - macOS It's a breaking change of 1.8.0, we'd better fix it since we've already realized it. If we cannot fix it, we should document the workaround. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
