Skip to content

Remove unused request body parameter from HTTP trigger (#704)#706

Merged
ziagham merged 2 commits intoflowsynx:masterfrom
mohin-io:issue-704-remove-requestbody
Nov 12, 2025
Merged

Remove unused request body parameter from HTTP trigger (#704)#706
ziagham merged 2 commits intoflowsynx:masterfrom
mohin-io:issue-704-remove-requestbody

Conversation

@mohin-io
Copy link
Copy Markdown
Contributor

Summary

  • remove the unused requestBody parameter from WorkflowHttpTriggerProcessor.ExecuteWorkflowAsync and the associated route handler so the class no longer carries dead code
  • update the HTTP listener registration to use a discard parameter and add focused XML documentation so the method's orchestration responsibilities are clear and consistent with existing style

Testing

  • dotnet test FlowSynx.sln (fails: installed SDK 8.0.415 cannot target net9.0; a newer .NET SDK is required)

Closes #704.

@mohin-io mohin-io requested review from a team as code owners November 12, 2025 00:04
@sonarqubecloud
Copy link
Copy Markdown

@ziagham ziagham merged commit d6ffd86 into flowsynx:master Nov 12, 2025
4 checks passed
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 this pull request may close these issues.

Remove unused method parameter requestBody in ExecuteWorkflowAsync

2 participants