You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@skywalking.apache.org by "heyanlong (via GitHub)" <gi...@apache.org> on 2024/06/12 08:09:00 UTC

[PR] Add TLS [skywalking-php]

heyanlong opened a new pull request, #112:
URL: https://github.com/apache/skywalking-php/pull/112

   Add `tls-roots` feature for use of trusted ca certificate for client.


-- 
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: notifications-unsubscribe@skywalking.apache.org

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


Re: [PR] Add TLS [skywalking-php]

Posted by "wu-sheng (via GitHub)" <gi...@apache.org>.
wu-sheng commented on PR #112:
URL: https://github.com/apache/skywalking-php/pull/112#issuecomment-2162377762

   What is this used about?


-- 
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: notifications-unsubscribe@skywalking.apache.org

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


Re: [PR] Add TLS [skywalking-php]

Posted by "jmjoy (via GitHub)" <gi...@apache.org>.
jmjoy commented on PR #112:
URL: https://github.com/apache/skywalking-php/pull/112#issuecomment-2162614837

   There are some issues with the source of PHP 8.2. Try again after a while? If it doesn't work, skip the CI of PHP 8.2 first.


-- 
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: notifications-unsubscribe@skywalking.apache.org

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


Re: [PR] Add TLS [skywalking-php]

Posted by "heyanlong (via GitHub)" <gi...@apache.org>.
heyanlong commented on PR #112:
URL: https://github.com/apache/skywalking-php/pull/112#issuecomment-2162383428

   For clients in other languages, the trusted CA root certificate will be automatically read by default for gRPC requests. For tonic, We need to add a feature to support this.


-- 
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: notifications-unsubscribe@skywalking.apache.org

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


Re: [PR] Add TLS [skywalking-php]

Posted by "wu-sheng (via GitHub)" <gi...@apache.org>.
wu-sheng commented on PR #112:
URL: https://github.com/apache/skywalking-php/pull/112#issuecomment-2162455889

   CI fails for some reason, you need to fix them first. Other parts, let's wait for @jmjoy 


-- 
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: notifications-unsubscribe@skywalking.apache.org

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