Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clarify gRPC insecure option #2476

Conversation

dyladan
Copy link
Member

@dyladan dyladan commented Apr 7, 2022

Fixes #2420

Changes

  • Clarify the insecure option for gRPC exporters

Related issues

@dyladan dyladan requested review from a team April 7, 2022 21:00
@dyladan dyladan force-pushed the grpc-insecure-clarification branch from 7891e93 to cafac36 Compare April 7, 2022 21:00
CHANGELOG.md Outdated Show resolved Hide resolved
@reyang reyang added the spec:protocol Related to the specification/protocol directory label Apr 7, 2022
specification/protocol/exporter.md Outdated Show resolved Hide resolved
dyladan and others added 2 commits April 8, 2022 11:00
@dyladan
Copy link
Member Author

dyladan commented Apr 11, 2022

Failed link check unrelated

@tigrannajaryan
Copy link
Member

Some links returned 502. Rerunning.

@reyang
Copy link
Member

reyang commented Apr 11, 2022

@dyladan "This branch is out-of-date with the base branch".

@dyladan
Copy link
Member Author

dyladan commented Apr 12, 2022

@reyang fixed and enabled modification by maintainers. Thanks

@tigrannajaryan tigrannajaryan merged commit feb75ef into open-telemetry:main Apr 12, 2022
@dyladan dyladan deleted the grpc-insecure-clarification branch April 12, 2022 15:01
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
Fixes open-telemetry#2420

## Changes

- Clarify the insecure option for gRPC exporters

Related issues

- open-telemetry/opentelemetry-js#2827
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec:protocol Related to the specification/protocol directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clarify TLS/Secure default behavior
8 participants