New Plugin: Discord Notifications

[GrimReaper 4739]

9 minutes ago, BillOatman said:

Same test please with debug on.  

Emby.Notifications.Discord102.dll.zip 55.37 kB · 1 download

Attached. All same, this time added an episode instead of a movie. And pls do note that Test message IS going through, as it did from the start.

embyserver.txt

 

Edited May 17, 2021 by GrimReaper76

[BillOatman 596]

9 minutes ago, GrimReaper76 said:

Attached. All same, this time added an episode instead of a movie. And pls do note that Test message IS going through, as it did from the start.

Yeah thats because its barfing trying to process whats in the queue from Emby, not talking to discord.

And again please ... tracking down the exact issue. As always, debug on.

 

Emby.Notifications.Discord103.dll.zip

[GrimReaper 4739]

New Media notification just came through now, for that last episode added.

Although it ain't enriched as @ryzilla's screenshot:

 

even though is marked in Plugin settings:

 

Edit: I think that's my hourly Playback Reporting plugin New Media notification.

Edited May 17, 2021 by GrimReaper76

[BillOatman 596]

1 minute ago, GrimReaper76 said:

New Media notification just came through now, for that last episode added.

Although it ain't enriched as @ryzilla's screenshot:

 

even though is marked in Plugin settings:

Probably because of the issue its having processing the queue from Emby.  We're getting there

[GrimReaper 4739]

7 minutes ago, BillOatman said:

Yeah thats because its barfing trying to process whats in the queue from Emby, not talking to discord.

And again please ... tracking down the exact issue. As always, debug on.

 

Emby.Notifications.Discord103.dll.zip 55.54 kB · 0 downloads

Here, same.

[BillOatman 596]

Just now, GrimReaper76 said:

Here, same.

No server log

[GrimReaper 4739]

Just now, BillOatman said:

No server log

Yeah, sorry. 

embyserver-63756871304.txt

[BillOatman 596]

11 minutes ago, GrimReaper76 said:

Yeah, sorry. 

It seems to be making some assumptions on what "richer" data exists that may not always be true.  Dialing in now what exactly the field is. Same, debug on please.
 

Emby.Notifications.Discord104.dll.zip

[GrimReaper 4739]

4 minutes ago, BillOatman said:

It seems to be making some assumptions on what "richer" data exists that may not always be true.  Dialing in now what exactly the field is. Same, debug on please.
 

Emby.Notifications.Discord104.dll.zip 55.72 kB · 0 downloads

That's some great progress. As ALL my metadata and artwork is handled by 3rd party app and only parsed by Emby, maybe it ain't writing something plugin is looking for.

embyserver.txt

[BillOatman 596]

13 minutes ago, GrimReaper76 said:

That's some great progress. As ALL my metadata and artwork is handled by 3rd party app and only parsed by Emby, maybe it ain't writing something plugin is looking for.

 

And same again please.  

Emby.Notifications.Discord105.dll.zip

[GrimReaper 4739]

3 minutes ago, BillOatman said:

And same again please.  

Emby.Notifications.Discord105.dll.zip 55.85 kB · 0 downloads

Running out of things to add. 

embyserver.txt

[BillOatman 596]

@GrimReaper76  What is your server name configured as for the plugin? And is ServerNameOverride true or false?

Edited May 17, 2021 by BillOatman

[GrimReaper 4739]

Just now, BillOatman said:

@GrimReaper76  What is your server name configured as for the plugin?

 

Maybe no space?

Lemme test.

[BillOatman 596]

3 minutes ago, GrimReaper76 said:

 

Maybe no space?

Lemme test.

Whats your emby server name itself?  And the thing to try would be unchecking "Replace "Emby Server with your server name" and trying

webhook name shouldnt matter as that seems to be working

Edited May 17, 2021 by BillOatman

[GrimReaper 4739]

Just now, BillOatman said:

Whats your server name itself?  And the thing to try would be unchecking "Replace "Emby Server with your server name" and trying

webhook name shouldnt matter as that seems to be working

Server name itself is BLADE. Same as I changed Webhook name to, now. Still no joy, test message goes through, new media crashes server.

embyserver-63756874552.txt

[BillOatman 596]

@GrimReaper76   uncheck "Replace Emby Server with your server name" and try, no need for server log on this one

[GrimReaper 4739]

6 minutes ago, BillOatman said:

@GrimReaper76   uncheck "Replace Emby Server with your server name" and try, no need for server log on this one

In process.

Edit: @BillOatman Man, you're awesome.  HUGE thanks for going through all this. I'd like to drop you a beer, is there a place I could do that? Thanks again.

Edited May 17, 2021 by GrimReaper76

[BillOatman 596]

@GrimReaper76 Sweet   Thanks for the offer, but maybe just help @ryzilla if he needs it.  He seems to be missing some GUID in his configuration.

I'll ask for some more help in a bit, I'll try and put some kind of fix in for the server name.

@Luke  It's doing this to get the server name, which he has set but apparently is coming through as null to the plugin.

private readonly IServerConfigurationManager _serverConfiguration;

public Notifier(ILogManager logManager, IJsonSerializer jsonSerializer, IServerConfigurationManager serverConfiguration, ...
_serverConfiguration = serverConfiguration;

 ServerConfiguration serverConfig = _serverConfiguration.Configuration;
 string serverName = options.ServerNameOverride ? serverConfig.ServerName : "Emby Server";  // Note:  options.ServerNameOverride is true

mediaAddedEmbed.embeds.First().title = $"{titleText} has been added to {serverName.Trim()}"; // serverName.Trim() throws null reference exception

Is that not correct any more?

Edited May 17, 2021 by BillOatman

[GrimReaper 4739]

5 minutes ago, BillOatman said:

@GrimReaper76 Sweet   Thanks for the offer, but maybe just help @ryzilla if he needs it.  He seems to be missing some GUID in his configuration.

I'll ask for some more help in a bit, I'll try and put some kind of fix in for the server name.

@Luke  It's doing this to get the server name, which he has set but apparently is coming through as null to the plugin.

private readonly IServerConfigurationManager _serverConfiguration;

public Notifier(ILogManager logManager, IJsonSerializer jsonSerializer, IServerConfigurationManager serverConfiguration, ...
_serverConfiguration = serverConfiguration;

 ServerConfiguration serverConfig = _serverConfiguration.Configuration;
 string serverName = options.ServerNameOverride ? serverConfig.ServerName : "Emby Server";  // Note:  options.ServerNameOverride is true

Is that not correct any more?

Sure, if he chimes in. I'm getting no more errors in log, so that's that. 

[BillOatman 596]

@GrimReaper76 Try this with "Replace Emby Server with your server name" checked again.  Debug log only if it fails please.  I suspect you won't get your server name but it should at least not fail with it checked. After Luke tells me whats wrong I'll try and make the server name work and see about getting this change into github.

Emby.Notifications.Discord107.dll.zip

[GrimReaper 4739]

28 minutes ago, BillOatman said:

@GrimReaper76 Try this with "Replace Emby Server with your server name" checked again.  Debug log only if it fails please.  I suspect you won't get your server name but it should at least not fail with it checked. After Luke tells me whats wrong I'll try and make the server name work and see about getting this change into github.

Emby.Notifications.Discord107.dll.zip 55.86 kB · 1 download

"Replace Emby Server with your server name" checked, content added, notification WITH Webhook name received, no errors thrown in log, all peachy. Great work, mate. 

[BillOatman 596]

Just now, GrimReaper76 said:

"Replace Emby Server with your server name" checked, content added, notification WITH Webhook name received, no errors thrown in log, all peachy. Great work, mate. 

Sweet thanks.  

[BillOatman 596]

On 5/13/2021 at 11:17 PM, rechigo said:

I am sorry for been updating this plugin, I just have not had the time nor motivation to work on it anymore, and I kinda forgot it even existed to be totally honest.. If anybody would like to fork the Github repo to try and fix it up & submit some pull requests, I (and it would appear the community members as well) would very much appreciate it. I still have the Github repo linked to a Discord webhook so I'll see whenever a pull request has been started and I'll be able to review them & publish releases to this thread & Github releases. This plugin desperately needs to be rewritten, I have very little knowledge in C# so the code isn't very readable to say the least, I believe this was my first project that I ever worked on with C#. 

 

tldr; I have been too busy & lacking in motivation to work on this plugin recently. if anyone is interested in maintaining this plugin, please let me know via PMs.

Hi:  I think I just put in a pull request to get it back functioning again.

[Legend1981 0]

last version emby beta 4.6.0.47-beta or all beta 4.6 the plugin Emby Discord Notifications 1.0.7.9 dont work  more people have this problem??????

[Luke 42077]

17 minutes ago, Legend1981 said:

last version emby beta 4.6.0.47-beta or all beta 4.6 the plugin Emby Discord Notifications 1.0.7.9 dont work  more people have this problem??????

Hi, the developer is investing his own time and energy into the plugin. Please help make it easier for him to help you by seeing how to report a problem. Thanks.