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 CoreWCF Instrumentation #1939

Open
marlon-tucker opened this issue Jul 4, 2024 · 1 comment
Open

Add CoreWCF Instrumentation #1939

marlon-tucker opened this issue Jul 4, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@marlon-tucker
Copy link

Component

None

Is your feature request related to a problem?

WCF instrumentation is already supported.

CoreWCF is the .NET Foundation supported framework designed to bring a level of support for WCF into the modern AspNetCore pipeline.

However, CoreWCF uses a different namespace to System.ServiceModel which means the existing WCF instrumentations can not be used.

What is the expected behavior?

A new instrumentation library is created specifically for CoreWCF. The Client channel instrumentation does not need to be implemented as CoreWCF is only server side.

The instrumentation should also be able to be used in conjunction with AspNetCore instrumentation, which will likely have created it's own Activity before the CoreWCF pipeline engages - so how the two activities interact remains to be seen.

Which alternative solutions or features have you considered?

There is no alternative at the moment.

Additional context

The majority of the existing WCF instrumentation can likely be copy pasted with just a few minor alterations to how things are reflected over and namespaces, plus removing any of the client channel infrastructure. The only major difference I can see will be how the existing Activity from AspNetCore is handled.

There is also question of supported runtimes, I see from the contributing guidelines projects are meant to target the same runtimes as the main OpenTelemetry project however CoreWCF does not target NET Framework so this won't be feasible.

@marlon-tucker marlon-tucker added the enhancement New feature or request label Jul 4, 2024
@Kielek
Copy link
Contributor

Kielek commented Jul 4, 2024

There is a plan to implement this feature natively in CoreWCF. My recommendation is to invest time directly in CoreWCF repository, instead of creating additional package.

Then, you type something like this tracerBuilder.AddSource("CoreWCF"); without including any external packages.

Ref: CoreWCF/CoreWCF#790

@birojnayak, please check this request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants