nlog/nlog.targets.redis

NLog Target for writing to Redis

nlog.targets.redis's Issues

Fix build of PRs

Something wrong with the webhook

Change DataType property to enum

Nuget/Project Name

We need a name to publish Nuget packages under, suggestions:

• NLog.Redis (https://www.nuget.org/packages/NLog.Redis/) - owned by richclement, failed to get in contact. Preferred.
• NLog.Targets.Redis (https://www.nuget.org/packages/NLog.Targets.Redis/) - owned by @diadistis, might require backwards compatibility.
• NLog.Target.Redis - breaks namespace conventions, might be too similar to the above suggestion.
• NLog.Contrib.Redis - wierd naming, unlike other NLog packages.

We might be able to petition Microsoft/Nuget to get contributor access to NLog.Redis.

Add SourceLink and deterministic build

Convert RedisTarget Parameters to use Layouts

With the move to .Net Core we can pull variables in from the appsettings.json files. So, for the RedisTarget, should all of the parameters also be using Layouts like the key?

Strong name code

Like all NLog libs, with the same key

fix assembly version to 1.0.0.0?

Merge support for multiple servers?

Maybe we should merge this one, before release

richclement/NLog.Redis#4

Update packagename, info, urls, readme

+move to version 1.0

Rename csproj to NLog.Targets.Redis

Change namespace to NLog.Redis?

Currently NLog.Targets

Update Nuget API Key

Regarding deployments we need a valid Nuget API key.

Add unit tests

The current unit tests are integration tests so they need Redis running.

That isn't really needed.

1. Make RedisTarget mockable
2. Create mock that writes into memory
3. Inject mocks in unit tests
4. Copy tests, keep needed, and change to unit test

Consider Embedding the SE.Redis Library

We should consider embedding the Redis library into the assembly - functionally this will reduce the API surface area. This would be similar to how Hangfire inlines the same package into Hangfire.Pro.Redis (sorry, only example I could think of).

Change Password, Db and Host to Layout

Remove .nuget references in .sln

rename repo to NLog.Targets.Redis?

Maybe we should rename this repo to NLog.Targets.Redis

That's more transparant I think.

(note: github will create a redirect for NLog.redis then)

Readme Config File Redis Port Wrong

change 3679 to 6379

Fix snupkg upload

• Manual upload
• fix script

The specified NuGet API key is invalid or has expired - Ver. 3.1.0

Believe the nuget-API-key needs to be updated:

• https://ci.appveyor.com/project/nlog/nlog-redis/builds/48075990

Maybe consider changing so NLog.Targets.Redis uses the same API-key as the other NLog-projects (But it also seems the Nuget-API-key for the other NLog-projects has also expired)

Fix line endings

e.g. see

from https://github.com/NLog/NLog.Redis/pull/16/files

Update nlog-project.org's targets page with new NLog.Redis link

The page https://nlog-project.org/config/?tab=targets still links to the old repo.

Release 3.0

How to configure the expiration time of this data included in redis in NLog.config file of asp.net?

I tried to add the expireSeconds field in the configuration file.

But I checked redis and found that ttl is still -1 instead of 60.

Add Nuget Badge to Readme

Update README with Name Change Explanation

Titles says it all.