Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Oct 9, 2025

Problem

The backend build was producing a deprecation warning during compilation:

warning CS0618: 'NpgsqlConnectionStringBuilder.TrustServerCertificate' is obsolete: 
'The TrustServerCertificate parameter is no longer needed and does nothing.'

This warning appeared at line 381 in Program.cs within the TryGetSafeConnectionInfo method, which is used for logging sanitized database connection information.

Solution

Removed the obsolete TrustServerCertificate property from the connection string builder output. According to the Npgsql library documentation, this parameter is no longer needed and has no effect.

Before:

return $"Host={builder.Host};Port={builder.Port};Database={builder.Database};SslMode={builder.SslMode};TrustServerCertificate={builder.TrustServerCertificate}";

After:

return $"Host={builder.Host};Port={builder.Port};Database={builder.Database};SslMode={builder.SslMode}";

Impact

  • ✅ Build now completes with 0 warnings and 0 errors
  • ✅ No functional changes to application behavior
  • ✅ The method continues to properly sanitize and log connection information
  • ✅ All essential connection parameters (Host, Port, Database, SslMode) are still included in the output

This is a minimal, surgical fix that eliminates build warnings without affecting any runtime functionality.

Fixes issue shown in the build logs screenshot.

Original prompt

This section details on the original issue you should resolve

<issue_title>Build warnings</issue_title>
<issue_description>Image</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #36


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@vercel
Copy link

vercel bot commented Oct 9, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
recall-ai Ready Ready Preview Comment Oct 9, 2025 4:55pm

Co-authored-by: christoph-codes <54452490+christoph-codes@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix build warnings in the project Fix build warning: Remove obsolete TrustServerCertificate property Oct 9, 2025
Copilot AI requested a review from christoph-codes October 9, 2025 16:55
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.

Build warnings

2 participants