refit icon indicating copy to clipboard operation
refit copied to clipboard

Add RestMethodInfo in HttpRequestMessage (Options or Properties)

Open Int32Overflow opened this issue 3 years ago • 1 comments

What kind of change does this PR introduce? Feature

What is the current behavior? No RestMethodInfo is available in the HttpMessageHandler

What is the new behavior? RestMethodInfo is now available in the HttpMessageHandler. Parameter-Name: RestMethodInfo { get; } = "Refit.RestMethodInfo";

What might this PR break? Nothing

Please check if the PR fulfills these requirements

  • [x] Tests for the changes have been added (for bug fixes / features)
  • [ ] Docs have been added / updated (for bug fixes / features)

Other information:

Int32Overflow avatar May 06 '22 08:05 Int32Overflow

Same as #1317

Int32Overflow avatar May 06 '22 10:05 Int32Overflow

My implementation of the same feature (#1317) was a little bit more complete with strongly typed HttpRequestOptionsKey and with updates to the README. Maybe you should also consider merging it.

0xced avatar Apr 12 '23 08:04 0xced

My implementation of the same feature (#1317) was a little bit more complete with strongly typed HttpRequestOptionsKey and with updates to the README. Maybe you should also consider merging it.

Thank you, we will take a look at this too, we're trying to get everything back in order and we will look at all the PRs in good time. Thank you for your efforts and patience.

ChrisPulman avatar Apr 12 '23 08:04 ChrisPulman

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

github-actions[bot] avatar Apr 27 '23 00:04 github-actions[bot]