Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add MySql support to the Certificate DB #1048

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rblackin
Copy link
Contributor

Adding support for the Certificate tables to be used on MySql.

@jstedfast
Copy link
Owner

This looks fine to me, but w/o unit tests I can't verify it.

You'll note that MimeKit.csproj has the following:

<Compile Remove="Cryptography\NpgsqlCertificateDatabase.cs" />

The reason for this, is again, the inability to write unit tests for PostgresSQL.

I would be happy to accept this PR, but I will likely need to remove it from the default build in the same way that Npgsql is being removed.

@jstedfast
Copy link
Owner

@rblackin any thoughts/preferences? Also, if you know of a good way to add unit tests that would be able to setup/connect to/make use of a MySQL database, I'd love to know because if that's something that you might know how to get working, then it would be the ideal situation because I would actually love to include this.

@jstedfast
Copy link
Owner

Helpful hint to myself later: maybe this guide would be useful? https://medium.com/@michaelmatere606/configuring-mysql-for-github-actions-a-step-by-step-guide-f49e0a7aeedc

@jstedfast jstedfast force-pushed the master branch 7 times, most recently from 4570379 to b0a9016 Compare October 16, 2024 01: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.

2 participants