-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Example run error #5652
Comments
This is may be fixed by #5132 However, notice that W3C baggage propagator would not handle such keys. |
I see |
What do you expect? |
As a new user of the framework, I had little knowledge of the w3c specification. I just want the functionality shown by example to work. Just example/namedtracer/main. go show usage is not correct. That bothers me |
With #5132 being merged, this issue appear to be solved. package main
import (
"fmt"
"os"
"go.opentelemetry.io/otel/attribute"
"go.opentelemetry.io/otel/baggage"
)
func main() {
fooKey := attribute.Key("ex.com/foo")
m0, _ := baggage.NewMemberRaw(string(fooKey), "foo1")
fmt.Fprintf(os.Stdout, "%#v", m0)
}
|
Description
example/namedtracer/main.go
An error in the file regarding the usage of baggage
The following code in the example does not create a new object and returns an error.
Because when the validateKeyChar method in baggage checks for '/', it returns an error.
Because safeKeyCharset in baggage doesn't contain '/'。
Expected behavior
Example run
The text was updated successfully, but these errors were encountered: