Skip to content

Why does WebProxyClient override Dispose to an empty method. #255

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

Closed
8ggmaker opened this issue Mar 26, 2022 · 7 comments · Fixed by #360
Closed

Why does WebProxyClient override Dispose to an empty method. #255

8ggmaker opened this issue Mar 26, 2022 · 7 comments · Fixed by #360

Comments

@8ggmaker
Copy link
Member

The WebProxyClient's Dispose method seems close over the ClientBase's dispose method, and also I call ServiceClient.Dispose every time, there are lots of HttpClient in HttpChannelFactory:
image

@MattB-msft
Copy link
Member

if your using .net core,
The issue your seeing is (we believe) unrelated to the dispose method, We believe this is a quirk of the WCF implementation in .net core.

Could you give us a bit more information on your use case? how are you using the client?

thanks

@8ggmaker
Copy link
Member Author

We now create a connection pool of ServiceClient, there is a parent client with its clones, when the pool is ready to delete, we dispose the parent and clones

@8ggmaker
Copy link
Member Author

8ggmaker commented May 15, 2022

@8ggmaker
Copy link
Member Author

@MattB-msft Could you help have a look at this issue, we are now using this sdk and will release at the end of this month, Thanks

@MattB-msft
Copy link
Member

We will look at optimizing that, however that code has been in production use across the Microsoft ecosystem since 2015 with very few reported issues of this type. There is some research going on regarding this and memory consumption patterns on our servers.

That said,
You should be fine to use it now.

We are keeping this item open while we research it.

@8ggmaker
Copy link
Member Author

Thanks for the information, by the way, we use .net core, I don't know whether the implementation of WCF in .net core is different

@MattB-msft
Copy link
Member

Lots of things are different in .net core unfortunately (or fortunately depending on view point :) )
The .net core team has recently released the rollup of the WCF work they have been doing with us and others... we are talking them about this currently.

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

Successfully merging a pull request may close this issue.

2 participants