Platform Evolution
Assessment run-2026-07-27-1 · 213 of 213 repos assessed
Repos / booking-download

booking-download

Stack of two ASP.NET Core APIs that import supplier bookings into Phenix: a Main API owning enrichment and the user-facing contract, and a Provider API owning raw supplier ingest (TBF, Trainline, NDC, TAA).

66 · amberlowloosely attachedOn-premZDD: needs workapi

Health score

66

0 to 100 across seven weighted factors; nothing is hidden. Green is 75 and above, amber 50 to 74, red below 50. How scoring works.


Runtime currency / 25
0
.NET Framework, out of mainstream support
Legacy transport / 15
6
Calls at least one legacy WCF endpoint
Test posture / 15
15
Good test coverage
Security hygiene / 15
15
No credentials found in source
Dangerous coupling / 15
15
No dangerous coupling detected
Activity / 10
10
Actively maintained
Deployment hygiene / 5
5
CI in place
Health over time
2026-07-24 (run-2026-07-24-1): 662026-07-27 (run-2026-07-27-1): 66
No change since 2026-07-24 (66 to 66).

Facts

Languagescsharp
FrameworksASP.NET Core 10, Serilog, FluentValidation, Polly
Runtimes.NET 10, .NET Framework 4.7.2 (contract packages)
Tests / CIgood / CI in place
Last commit2026-07-24
OwnerOwner unknown (mapping to be curated)
DomainBookings and fulfilment (system generated; curate in repo-owners.json to override)
Product areaPhenix
HostedOn-prem Deployed to our own machines
  • Release "\flights-stream\booking-download\booking-download-api": Staging: deployment-group phase (agents on our own machines)
  • Release "\flights-stream\booking-download\booking-download-api": Production: deployment-group phase (agents on our own machines)
  • Release "\flights-stream\booking-download\booking-download-api": Training: deployment-group phase (agents on our own machines)
  • Release "\flights-stream\booking-download\booking-download-api": DR: deployment-group phase (agents on our own machines)
  • Release "\flights-stream\booking-download\booking-download-api": QA: deployment-group phase (agents on our own machines)
Releases3 release pipeline(s): \flights-stream\booking-download\booking-download-api; \flights-stream\booking-download\booking-download-provider-api; \flights-stream\booking-download\zz obsolete booking-download-ui
Integrationrest, wcf
SourceOpen in Azure DevOps

Risk flags

Each flag links to its glossary definition.

Dependencies in context

Laying out the graph…
greenamberreddata store (square, dashed link)Node size follows fan-in. Click any node to open it.

Entanglement

loosely attached Bound into the core by 1 mechanism(s).

  • Speaks WCF (client)

Detaches when its WCF surface gains a modern interface or its WCF calls move to one.

Migration playbooks

Generated from this repo's assessment data: what is feasible, what blocks it, and the steps in order. Ratings: green means actionable now, amber means sequence other work first, red means blocked, grey means not applicable.

ZDD adoption

Needs work

Feasible through the Workflow Engine's rolling drain, but real preparation is needed: dual-version discipline, better tests, or schema care before at-will release is sensible. Legacy runtime: the rolling drain works (stealth releases prove it) but relies on fix-forward risk rather than dual-version safety.

Dependencies and gotchas
  • 4 system(s) depend on this repo; during a rolling deploy they will talk to old and new versions at once, so responses must stay compatible
  • Consumes WCF contract packages: never take a breaking client bump mid-feature, and treat obsolete warnings as migration debt
Steps
  1. Adopt the four-stage discipline from the ZDD engineering standard: pre-deployment compatibility, deployment, feature activation behind a flag, post-release cleanup.
  2. Make every change dual-version safe: backwards-compatible APIs (new major version for breaking wire changes), expand and contract for any schema change, and consumers tolerant of unknown fields.
  3. Audit serialized data: anything in Redis, queues or shared caches must deserialize cleanly across adjacent versions.
  4. Wrap behaviour changes in feature flags so code deploys dark and release becomes a business decision, not a deployment event.
  5. Route deployments through the Workflow Engine's automated drain, deploy, health-check, return cycle (the path proven by the first automated production run in mid-2026).
  6. Verify with area-scoped regression rather than the whole-platform cycle, and watch error rates after rollout with automated rollback on alert.

SQL Server exit

Feasible after replatforming (1 owned database)

The database is solely owned so no other system has to move, but the runtime is end-of-life; sequence the replatform first, then the engine swap.

Steps
  1. Inventory the schema and query surface of sqlserver:GidsExtensions: tables, stored procedures, triggers, cross-database joins and anything else engine-specific.
  2. Confirm the target: PostgreSQL (relational model; keep transactions and joins, swap the engine).
  3. Replatform to modern .NET first (see the recommended actions): current PostgreSQL and MongoDB drivers and EF Core providers target modern .NET, and doing one risky change at a time is cheaper than doing two.
  4. Data access is not through an ORM, so port the SQL by hand: translate T-SQL constructs, replace stored procedures with application code or target-native equivalents, and add integration tests around the queries that matter first.
  5. Stand up the target alongside SQL Server, backfill with an ETL pass (pgloader or an export/import job), then dual-write or replicate until parity is proven.
  6. Move reads to the target, watch correctness and latency, then cut writes over and keep SQL Server read-only until confidence is earned.
  7. Decommission the SQL Server database and remove the connection strings and firewall paths that pointed at it.

Containerisation

Feasible after replatforming

Appropriate, but the .NET Framework runtime has to go first; container work rides along with the replatform rather than preceding it.

Blockers
  • .NET Framework only runs on Windows; practical (Linux) containers need modern .NET
Steps
  1. Replatform to modern .NET first (see the recommended actions). Containerising then becomes routine.
  2. Add a Dockerfile and make the app read all configuration from environment variables and the secrets manager, never from machine-local files.
  3. Log to stdout and expose a health check endpoint so ECS can supervise it.
  4. Adopt the shared ci.yml pipeline template with the container parameters (awsRegion, containerAfterBuild, ecsTerraformRepo), which is exactly how the estate's existing AWS services deploy.
  5. Define the service in the Terraform repo, deploy to a non-production environment, then shift traffic and disarm the deployment-group release pipeline.

Recommended actions

Replatform booking-download to modern .NET

Runs on .NET 10, .NET Framework 4.7.2 (contract packages), out of mainstream support. No pinning dependency blocks it; sequence by fan-in (providers before consumers).

Impact: Unblocks 4 dependent system(s)

proposed

Assessment notes

booking-download

Purpose

Imports supplier bookings into Phenix. The Main API (TC.BookingDownload.API) handles enrichment and the user-facing contract; the Provider API (TC.BookingDownload.Provider.API) handles raw supplier ingest for TBF, Trainline, NDC and TAA sources. Contract/client NuGet packages are consumed by Phenix and other services.

Tech stack

  • C# on .NET 10 for all services; contract packages multi-target net472;netstandard2.0 for backwards compatibility.
  • ASP.NET Core with API versioning, Swashbuckle, FluentValidation, Polly, Serilog (with New Relic enrichment), PhenixAuth authentication.
  • Heavy use of internal TC libraries: ConnectionBroker (Api/Mongo/Redis/Sql/Wcf), TC.Cache.Specialised, Phenix ApplicationFramework, various Phenix service clients.

Deployment

release_count 3. Build pipelines: booking-download-api, booking-download-provider-api, booking-download-ui (plus an obsolete UI pipeline). Release pipelines under \flights-stream\booking-download\. Repo contains four Azure Pipelines YAML files (CI plus NuGet package publishing per API). No UI code remains in the repo — the UI pipeline is marked obsolete.

Dependencies

  • Internal: client/contract packages for destination management, exchange rate, tax, linking, Trainline events, company info, user/contact-centre/flights services (several without a confirmed repo mapping).
  • External: Trainline, NDC airline content, TAA feed (via AWS S3/Mongo utilities), AWS Cognito auth, New Relic APM.

Data stores

  • MongoDB (bookingdownloadApi_logs logging DB; TAA JSON store via TaaJsonToMongo utility).
  • SQL Server via GidsExtensionsDatabase SQL project and System.Data.SqlClient/TC.ConnectionBroker.Sql.

Integration points

  • REST APIs (versioned, Swagger) for both Main and Provider APIs; publishes NuGet client/contract packages.
  • WCF client bindings (System.ServiceModel Http/NetTcp/Duplex) to legacy Phenix services.

Code health

Very active (last commit 2026-07-24). Extensive test projects across API, managers, providers and shared libraries; StyleCop/rulesets enforced. Well-documented README covering solutions, CI and contract-stability rules.

Modernisation risks

Low overall: current .NET 10, good tests and CI. Residual risks: net472 contract packages tie it to .NET Framework consumers (Phenix), WCF client dependencies on legacy services, and a SQL database project (GidsExtensionsDatabase) that may be shared with the GIDS estate.