New Plugin: Discord Notifications

[grafxxx]

Yes I thought as @GrimReaper76was posting about 4.6.0.50 that this is the new realease. just looked on the website and can not find it for debian. still 4.5.4.0 and my current beta so what was he testing?

[BillOatman]

3 minutes ago, grafxxx said:

Yes I thought as @GrimReaper76was posting about 4.6.0.50 that this is the new realease. just looked on the website and can not find it for debian. still 4.5.4.0 and my current beta so what was he testing?

I see this, not sure how quickly other platforms roll out.

Emby Server
Windows
Stable: v4.6.0.50 (2021/05/19)

[grafxxx]

ok have to wait then and test again as soon debian version is available.

[BillOatman]

@rechigo  I submitted another pull request for todays emby server release.

[GrimReaper]

39 minutes ago, BillOatman said:

@Luke This is how it is populating images ... should anything have changed in this server/library release for this (the exception message is not in the log)?

                           // populate images
                            if (item.HasImage(ImageType.Primary))
                            {
                                string imageUrl = "";

                                if (!item.GetImageInfo(ImageType.Primary, 0).IsLocalFile)
                                {
                                    imageUrl = item.GetImagePath(ImageType.Primary);
                                }
                                else if (serverConfig.EnableRemoteAccess == true && !options.ExcludeExternalServerLinks) // in the future we can proxy images through memester server if people want to hide their server address
                                {
                                    imageUrl = $"{sysInfo.WanAddress}/emby/Items/{itemId}/Images/Primary";
                                }
                                else
                                {
                                    string localPath = item.GetImagePath(ImageType.Primary);

                                    try
                                    {
                                        ImageServiceResponse response = MemesterServiceHelper.UploadImage(localPath, _jsonSerializer);
                                        imageUrl = response.filePath;
                                    }
                                    catch (Exception e)
                                    {
                                        _logger.ErrorException("Failed to proxy image", e);
                                    }
                                }

                                mediaAddedEmbed.embeds.First().thumbnail = new Thumbnail
                                {
                                    url = imageUrl
                                };
                            }

 

Sorted.

 

Looking at the code you posted, Remote Access needed to be enabled and port forwarded. Kinda like images get pulled instead of pushed.

 

Edit: And one for Movies. So, all sorted. 

 

Edited May 19, 2021 by GrimReaper76

[BillOatman]

What's weird @GrimReaper76 is I didn't change the code.  So something in the server changed on the upgrade?

But glad you figured it out!  I submitted that change to the owner so there should ne a official release soon.

Edited May 20, 2021 by BillOatman

[samuelqwe]

8 minutes ago, BillOatman said:

What's weird @GrimReaper76 is I didn't change the code.  So something in the server changed on the upgrade?

I think it’s just that before Discord was trying to access the images but couldn’t because his server wasn’t remotely accessible. Once he enabled remote access, Discord could finally reach the server and get the images.

[GrimReaper]

37 minutes ago, BillOatman said:

What's weird @GrimReaper76 is I didn't change the code.  So something in the server changed on the upgrade?

But glad you figured it out!  I submitted that change to the owner so there should ne a official release soon.

Nah, it was on my end (although not something I would think at first), and kept Remote Access disabled on test 4.6.0.50 server as there was no need for it, and by chance you posted that extract. I envisioned the notification and artwork to get pushed to Discord, looks like artwork gets pulled and Remote access is required. 

32 minutes ago, samuelqwe said:

I think it’s just that before Discord was trying to access the images but couldn’t because his server wasn’t remotely accessible. Once he enabled remote access, Discord could finally reach the server and get the images.

Yup, sounds about right. 

[Luke]

Is this all good now on 4.6?

[BillOatman]

37 minutes ago, Luke said:

Is this all good now on 4.6?

Yeah seems that way thanks.  Just waiting for the github update and "official" release.  Thanks!

[rechigo]

6 hours ago, BillOatman said:

@rechigo  I submitted another pull request for todays emby server release.

I checked the Github and I did not see any pull requests open. I looked for your fork and was unable to find it in the list (privated repo?)

I noticed some people were having trouble getting images embedded into Discord. If you have strict firewall rules, your firewall may be blocking requests to the image proxy (https://i.memester.xyz). I believe someone here had that issue before and they were able to resolve it with a new firewall rule. It's also possible that the server I'm running the image proxy on was off for a couple of hours for some maintenance recently. This image proxy is used under two circumstances: 

1) Remote access is disabled on the server

2) You have the ExcludeExternalServerLinks option checked on the config page (Hides direct links to server)

[kelly.clubb]

Updated to latest stable (it was newer than the version for my beta so it worked out). Updated to the latest discord notifcations dll and it's now fully working for me. Thank you for all of those that put work into making this work.

[grafxxx]

Great to hear that..Now I hope for the debian release so I can update... Thank you @BillOatman @GrimReaper76....

[kelly.clubb]

Have a friend that is in the same boat, hopefully it releases soon.

[BillOatman]

10 hours ago, rechigo said:

I checked the Github and I did not see any pull requests open. I looked for your fork and was unable to find it in the list (privated repo?)

I noticed some people were having trouble getting images embedded into Discord. If you have strict firewall rules, your firewall may be blocking requests to the image proxy (https://i.memester.xyz). I believe someone here had that issue before and they were able to resolve it with a new firewall rule. It's also possible that the server I'm running the image proxy on was off for a couple of hours for some maintenance recently. This image proxy is used under two circumstances: 

1) Remote access is disabled on the server

2) You have the ExcludeExternalServerLinks option checked on the config page (Hides direct links to server)

Weird.  Here are the 2 files that changed.  Set the version however you think is best, I just did that to keep them separate.

I also tried to do the pull request again, have a look.

Emby.Notifications.Discord.zip

Edited May 20, 2021 by BillOatman

[BillOatman]

@GrimReaper76 Are you getting your server name in the discord notification or "Emby Server"?

[GrimReaper]

27 minutes ago, BillOatman said:

@GrimReaper76 Are you getting your server name in the discord notification or "Emby Server"?

Yeah, all peachy.

[ryzilla]

Ive updated to the latest non beta and all is working as intended well done team!

[rechigo]

@BillOatmanIs it alright if I distribute builds for .48 beta or was there a reason it was build against .50 (stable)? Beta users don't actually get the stable release (.50), so they aren't able to run the plugin because the SDK is for the newer version

[BillOatman]

4 minutes ago, rechigo said:

@BillOatmanIs it alright if I distribute builds for .48 beta or was there a reason it was build against .50 (stable)? Beta users don't actually get the stable release (.50), so they aren't able to run the plugin because the SDK is for the newer version

It wasn't working correctly with the previous build so when Luke released that one I switched to it and things started working again.
Its up to you what you release with    I'd build with the latest and just ask beta users to get the stable  

[pir8radio]

so what is the latest version that works?     I have the one posted in this post: 

 

 

however its still not working for me with these settings...   and the settings look like something is up some of the library options say "Header" ahead of the actual emby library name.    I'm on latest beta.

 

 

 

 

Edited May 21, 2021 by pir8radio

[kelly.clubb]

1 hour ago, BillOatman said:

It wasn't working correctly with the previous build so when Luke released that one I switched to it and things started working again.
Its up to you what you release with    I'd build with the latest and just ask beta users to get the stable  

I also went from beta to stable since it was technically a uptick in version number so I'm now on .60 stable and all works for the plugin. Windows build.

[grafxxx]

So I'm now on the new stable debian version, with the latest plugin but I still don't get enriched movie or shows. Instead now I get the notification twice and one time it is white and the other blue. I attached the screenshot. Can you please help me @BillOatman? Since moving to the stable emby server crashes when scanning the library (crash at around 90%).

 

embyserver.txt

Edited May 21, 2021 by grafxxx

[BaukeZwart]

An other variation here, I'm getting enriched data only without the external links. 

Emby 4.6.0.50, plugin version 11112.
Synology OS, docker container.

embyserver.txt

Edited May 21, 2021 by BaukeZwart
Log added

[grafxxx]

I think it was the plugin because when I removed it, the scan was fine and the server did not crash. @BillOatman I attached the log. Do I need to change something in my setup?

embyserver-63757185567.txt embyserver-63757185819.txt