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

Add support for Blazor WASM #170

Open
dbheale opened this issue Jan 20, 2022 · 5 comments
Open

Add support for Blazor WASM #170

dbheale opened this issue Jan 20, 2022 · 5 comments
Labels
type: enhancement New feature or request

Comments

@dbheale
Copy link

dbheale commented Jan 20, 2022

Hello!

I'm working on multiple Blazor WASM applications expecting to go live mid-year and I would love to use Honeycomb.oi as a collector. Unfortunately, there's no runtime pack for it.

Is this something that would be considered?

Thank you!

@dbheale dbheale added the type: enhancement New feature or request label Jan 20, 2022
@cartermp cartermp added this to the GA Release milestone Jan 21, 2022
@cartermp
Copy link
Member

Tentatively adding to our GA milestone for now. We'll need to look into if we need to add the right stuff only in this SDK distribution, or also/only in the upstream vendor-neutral SDK. Either way it shouldn't be a terribly large lift, just requires some investigation since Blazor WASM projects are a little special w.r.t. dependencies.

@MikeGoldsmith MikeGoldsmith self-assigned this Sep 15, 2022
@cartermp
Copy link
Member

FYI @dbheale this should work today - we're adding an example here: #251

@cartermp
Copy link
Member

Plopping this link here: open-telemetry/opentelemetry-dotnet#2816

There are likely to be other issues with blazor-wasm even when an upstream fix lands and we take it. So while we should be able to make our SDK distro work with blazor-wasm, there may be further caveats.

@dbheale
Copy link
Author

dbheale commented Sep 21, 2022

@cartermp Awesome, I'll check it out soon!

@cartermp
Copy link
Member

Yeah, another quick update:

  • It's possible to use this distro to write stuff to the console, but that's only because it uses a simple span processor (i.e., compute and export each span as it's created)
  • It's not possible to configure a batch span processor since blazor is single-threaded. Which means the only way to use it would result in terrible performance for all but very simple apps
  • Furthermore, there's quirks in actually calling the honeycomb API from the client. You'd need to configure a CORS policy.

All-in-all, I'd say Blazor WASM isn't quite ready as an application type for OpenTelemetry just yet. We did work on our end to ensure that you can indeed use our SDK distribution with a Blazor app, but these issues in Blazor are unavoidable for now.

If you're interested in setting up some more machinery, through the magic of JS Interop you can actually configure an app to send to honeycomb with this project: https://github.com/martinjt/otel-blazor-exporter

But this is the experimental realm, so we're likely to consider our SDK as GA without a good end-to-end experience for Blazor WASM.

@MikeGoldsmith MikeGoldsmith removed this from the GA Release milestone Sep 21, 2022
@MikeGoldsmith MikeGoldsmith removed their assignment Sep 26, 2022
@MikeGoldsmith MikeGoldsmith changed the title Blazor WASM Add support for Blazor WASM Sep 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants