[GitHub] [skywalking] BFergerson commented on pull request #7131: Support authentication for log report channel.

[GitBox <gi...@apache.org>]

BFergerson commented on pull request #7131:
URL: https://github.com/apache/skywalking/pull/7131#issuecomment-864083931


   https://github.com/apache/skywalking/issues/7040 was marked as a duplicate of https://github.com/apache/skywalking/issues/6664 and https://github.com/apache/skywalking/issues/6664 is closed by this. I would think that means this should also fix the issue of the lack of TLS encryption mentioned in https://github.com/apache/skywalking/issues/7040 but this PR doesn't seem to do that.
   
   Also, I'd like to mention that I find it a bit odd that the progress of the primary SkyWalking project is being slowed down by secondary projects like the sidecar. I would think that the primary project sets the pace and the secondary projects would need to keep up to stay functional. Here it seems that there is progress that could be made with SkyWalking but due to issues with a secondary project, that progress is being put on hold.
   
   I believe you know what's best though so feel free to disregard my opinion.


-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org