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 OperationDescription.SyncMethod, BeginMethod, and EndMethod #5132

Open
t-palanis opened this issue Apr 27, 2023 · 4 comments
Open

Add OperationDescription.SyncMethod, BeginMethod, and EndMethod #5132

t-palanis opened this issue Apr 27, 2023 · 4 comments
Assignees
Labels

Comments

@t-palanis
Copy link

t-palanis commented Apr 27, 2023

Describe the bug
Hi! We are migrating our legacy application in ,Net framework to .NET core and noticed that
OperationDescription.SyncMethod, BeginMethod, and EndMethod are missing from the .NET Core API but are available in the .NET framework version

Similar request to #4549. Would it be possible to expose these?

cc @mconnew

@mconnew
Copy link
Member

mconnew commented Apr 27, 2023

We're locked in on the api surface that will be released for the upcoming 6.0 release. We can certainly add these, but it will be a while before they would be available to use. Can you describe a bit more about how you are using these properties? With the timing, if there's an alternative way to achieve your goal it might be better to modify your code to unblock you sooner.

@t-palanis
Copy link
Author

t-palanis commented May 9, 2023

@mconnew Thanks for the reply! From legacy, we have existing code to read custom attributes in service synchronization method(Syncmethod) via ContractDescription.Operations, currently unable to do so in .NET 6/7.

We don't have an immediate need and if this could be available in next few months, that would be helpful

@mconnew
Copy link
Member

mconnew commented May 9, 2023

No , the async nature of the operation is irrelevant when sent across the network. A client can call APM, Sync, or Task and the server can be any one of the three (in wcf) or the two for CoreWCF.
I replied to the wrong issue about something unrelated.

@HongGit HongGit assigned imcarolwang and mconnew and unassigned mconnew May 18, 2023
@HongGit
Copy link
Contributor

HongGit commented May 18, 2023

@imcarolwang can you please see if you could add these? In addition, please add tests for these APIs as well.

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

No branches or pull requests

4 participants