Skip to content

Simplify .NET provider flow - #26

Open
Nisheet Jain (nisheetjain) wants to merge 4 commits into
mainfrom
simplify-dotnet-provider-flow
Open

Nisheet Jain (nisheetjain) wants to merge 4 commits into
mainfrom
simplify-dotnet-provider-flow

Conversation

@nisheetjain

Copy link
Copy Markdown
Collaborator

Summary

  • replace dispatch, registry, envelope-parser, and outcome-mapper layers with direct Azure Function orchestration
  • add a typed PhoneProviderBase transport so providers map POCO responses instead of handling HttpResponseMessage
  • move credential acquisition into providers and use a single expiring in-memory credential cache
  • preserve JWE validation, correlation IDs, PII-safe logging, provider mappings, and endpoint behavior

Validation

  • 170 .NET tests pass
  • .NET build succeeds with zero warnings and errors

Replace the dispatch and response-parser layers with direct Azure Function orchestration and a typed provider transport base. Providers now own protocol translation and credential acquisition while shared HTTP, timeout, JSON, and logging behavior stays centralized.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 0769c5c6-97a7-4645-ab83-5cda60c73b16
Let JweDecryptor own RSA key loading and deserialize authenticated plaintext directly into typed payloads. Remove the key-provider abstraction and raw JSON traversal, and update package validation for the new provider base.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 0769c5c6-97a7-4645-ab83-5cda60c73b16
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 0769c5c6-97a7-4645-ab83-5cda60c73b16
Comment thread dotnet/Src/Providers/SopranoProvider.cs
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 0769c5c6-97a7-4645-ab83-5cda60c73b16
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.

2 participants