tc-web-details-api
.NET 8 API serving Travel Counsellor website details (profiles, blogs, offers, testimonials, videos, social media) for the public leisure/business websites, with a MyTC sync service.
Health score
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.
Facts
| Languages | csharp |
| Frameworks | ASP.NET Core 8, Entity Framework Core 8 |
| Runtimes | .NET 8 |
| Tests / CI | good / CI in place |
| Last commit | 2026-07-20 |
| Owner | Owner unknown (mapping to be curated) |
| Domain | Public web (system generated; curate in repo-owners.json to override) |
| Product area | Shared platform |
| Hosted | On-prem Deployed to our own machines
|
| Releases | 2 release pipeline(s): \marketing\web-details\tc-web-details-api; \marketing\web-details\tc-web-details-api stealth |
| Integration | rest, wcf, sqlserver, redis |
| Source | Open in Azure DevOps |
Dependencies in context
Entanglement
kernel Bound into the core by 2 mechanism(s).
- Pinning dependency: phenix-feature-service
- Speaks WCF (client)
Detaches when its pinning dependencies are cut or multi-targeted (phenix-feature-service); and 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
No excuseNothing technical is holding this repo on the three-week release. It either already deploys without downtime (AWS) or is a modern, well-tested service the automated stealth path can carry today. Modern runtime, good tests, no hosted legacy transport: the automated stealth path can carry it today. Calls legacy WCF endpoints as a client; stay dual-version safe on client package upgrades.
- 2 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
- Serialized state or messages in flight (Redis or queues): new code must deserialize objects written by the previous version
- Feature flags come from phenix-feature-service (WCF-only): flag reads themselves are a legacy dependency to watch
- Adopt the four-stage discipline from the ZDD engineering standard: pre-deployment compatibility, deployment, feature activation behind a flag, post-release cleanup.
- 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.
- Audit serialized data: anything in Redis, queues or shared caches must deserialize cleanly across adjacent versions.
- Wrap behaviour changes in feature flags so code deploys dark and release becomes a business decision, not a deployment event.
- 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).
- 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 now (1 owned database)The database is solely owned by this system, so the migration is self-contained. Recommended target: PostgreSQL.
- Inventory the schema and query surface of sqlserver:TravelCounsellorWebDetail: tables, stored procedures, triggers, cross-database joins and anything else engine-specific.
- Confirm the target: PostgreSQL (relational model; keep transactions and joins, swap the engine).
- The repo already uses Entity Framework: swap the provider (Npgsql for PostgreSQL) behind the existing model, then burn down the provider-specific differences the test suite finds.
- 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.
- Move reads to the target, watch correctness and latency, then cut writes over and keep SQL Server read-only until confidence is earned.
- Decommission the SQL Server database and remove the connection strings and firewall paths that pointed at it.
Containerisation
Ready to containeriseModern runtime, no hosted legacy transport: this service can follow the estate's proven container path (Dockerfile, shared pipeline template, ECS via Terraform).
- Add a Dockerfile and make the app read all configuration from environment variables and the secrets manager, never from machine-local files.
- Log to stdout and expose a health check endpoint so ECS can supervise it.
- 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.
- 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
Rotate the credentials committed to tc-web-details-api
Risk flags indicate credentials or keys in source (plaintext-credentials). Rotate now and enable secret scanning in CI; this is independent of all other work.
Impact: Closes a live security exposure at near-zero cost
proposedAssessment notes
tc-web-details-api
Purpose
API behind the public Travel Counsellors websites' per-counsellor detail pages: web profiles, blogs (with images/videos), offers, testimonials, tags and social-media links, plus image upload to an FTP file server. A background sync service pushes changes to the MyTC app via the public WebAPI service.
Tech stack
C# on .NET 8 (upgraded from the .NET Core 3.1 the README still describes), ASP.NET Core with API versioning and Swashbuckle, EF Core 8 with a scaffolded TravelCounsellorWebDetailContext, PhenixAuthLib plus TC.Cognito auth, API-key claims for known callers, FluentFTP, SixLabors.ImageSharp, TC.Cache with Redis storage, Nerdbank.GitVersioning and StyleCop/rulesets.
Deployment
prod_application, release_count 2. One YAML build pipeline (azure-pipelines.yml using shared azure-pipelines-templates, .NET 8 SDK) feeding release pipelines \marketing\web-details\tc-web-details-api and its stealth variant. The pipeline also archives and publishes a TravelCounsellorWebDetailDatabase SQL project artifact from db/, so the database schema ships with the API.
Dependencies
Internal packages: TC.ConnectionBroker.* (Sql, Api, Wcf, Redis — tc-connectionbroker), TC.Exceptions.* (tc-exceptions), TC.Phenix.CompanyInfo (company-info-api), TC.Public.WebAPI.Service.Api.Client/ServiceContracts (public-api), TC.Cache.Specialised.PhenixFeature (phenix-feature-service). External: AWS Cognito authentication, an FTP file server for image assets, New Relic APM (Live config).
Data stores
SQL Server database TravelCounsellorWebDetail via ConnectionBroker connection WebDetail (EF Core), with the dacpac-style DB project in-repo. Redis as cache storage (CacheStorage=redis).
Integration points
Inbound REST with API-key/JWT auth (known callers include Public Service, Contact Centre Email Server, Documentation, LeisureApi, TC Sync Service). Outbound: WCF call to the public WebAPI service for MyTC dataset sync, REST to company-info/feature services, FTP for image storage.
Code health
Eight test projects (unit tests across command/query/mappers/entities/auth plus SpecFlow integration tests) — good coverage by estate standards. Actively developed (last commit 2026-07-20). README is reasonable but stale on SDK version.
Modernisation risks
Low: modern .NET 8 with YAML CI and tests. Residual issues: FTP credentials stored in plaintext in appsettings.json/appsettings.Live.json (same password across environments), a legacy WCF client for MyTC sync, and the website-detail database is also touched by other callers via the API-key list, hinting at shared-DB coupling.