-
Notifications
You must be signed in to change notification settings - Fork 148
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
TimeStamp column nullable or not #541
Comments
ckadluba
added a commit
to ckadluba/serilog-sinks-mssqlserver
that referenced
this issue
Sep 26, 2024
The Timestamp column is nullable by default but the README.md stated the opposite. This was fixed now.
github-merge-queue bot
pushed a commit
that referenced
this issue
Sep 26, 2024
Fixed issue #541 in README.md
Merged
ckadluba
added a commit
that referenced
this issue
Oct 18, 2024
* Fixed issue #580: Removed deprecated transitive dependency on Microsoft.NETCore.Targets by removing runtime identifier (thanks to @david-brink-talogy) * Fixed issues #540 and #541 in README * Added performance tests including a GitHub actions workflow * Minor cleanups and fixes
Merged
ckadluba
added a commit
that referenced
this issue
Oct 18, 2024
* Fixed issue #580: Removed deprecated transitive dependency on Microsoft.NETCore.Targets by removing runtime identifier (thanks to @david-brink-talogy) * Fixed issues #540 and #541 in README * Added performance tests including a GitHub actions workflow * Minor cleanups and fixes
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
If I use the
"autoCreateSqlDatabase": true
setting, the[TimeStamp]
column is created asNULL
, although the readme says it'sNOT NULL
here.The auto created table should match the readme.
.NET 8
OS: Windows 10
builder.Host.UseSerilog((context, services, loggerConfiguration) => loggerConfiguration.ReadFrom.Configuration(context.Configuration));
Just start the app to let the log table creation happen.
The text was updated successfully, but these errors were encountered: