grafxxx 4 Posted May 19, 2021 Share Posted May 19, 2021 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? Link to comment Share on other sites More sharing options...
BillOatman 492 Posted May 19, 2021 Share Posted May 19, 2021 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) Link to comment Share on other sites More sharing options...
grafxxx 4 Posted May 19, 2021 Share Posted May 19, 2021 ok have to wait then and test again as soon debian version is available. Link to comment Share on other sites More sharing options...
BillOatman 492 Posted May 19, 2021 Share Posted May 19, 2021 @rechigo I submitted another pull request for todays emby server release. Link to comment Share on other sites More sharing options...
GrimReaper 3226 Posted May 19, 2021 Share Posted May 19, 2021 (edited) 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 1 Link to comment Share on other sites More sharing options...
BillOatman 492 Posted May 20, 2021 Share Posted May 20, 2021 (edited) 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 1 Link to comment Share on other sites More sharing options...
samuelqwe 313 Posted May 20, 2021 Share Posted May 20, 2021 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. 1 Link to comment Share on other sites More sharing options...
GrimReaper 3226 Posted May 20, 2021 Share Posted May 20, 2021 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. Link to comment Share on other sites More sharing options...
Luke 36888 Posted May 20, 2021 Share Posted May 20, 2021 Is this all good now on 4.6? Link to comment Share on other sites More sharing options...
BillOatman 492 Posted May 20, 2021 Share Posted May 20, 2021 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! 1 Link to comment Share on other sites More sharing options...
rechigo 293 Posted May 20, 2021 Author Share Posted May 20, 2021 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) 1 Link to comment Share on other sites More sharing options...
kelly.clubb 8 Posted May 20, 2021 Share Posted May 20, 2021 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. 1 Link to comment Share on other sites More sharing options...
grafxxx 4 Posted May 20, 2021 Share Posted May 20, 2021 Great to hear that..Now I hope for the debian release so I can update... Thank you @BillOatman @GrimReaper76.... Link to comment Share on other sites More sharing options...
kelly.clubb 8 Posted May 20, 2021 Share Posted May 20, 2021 Have a friend that is in the same boat, hopefully it releases soon. Link to comment Share on other sites More sharing options...
BillOatman 492 Posted May 20, 2021 Share Posted May 20, 2021 (edited) 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 Link to comment Share on other sites More sharing options...
BillOatman 492 Posted May 20, 2021 Share Posted May 20, 2021 @GrimReaper76 Are you getting your server name in the discord notification or "Emby Server"? Link to comment Share on other sites More sharing options...
GrimReaper 3226 Posted May 20, 2021 Share Posted May 20, 2021 27 minutes ago, BillOatman said: @GrimReaper76 Are you getting your server name in the discord notification or "Emby Server"? Yeah, all peachy. 1 Link to comment Share on other sites More sharing options...
ryzilla 11 Posted May 20, 2021 Share Posted May 20, 2021 Ive updated to the latest non beta and all is working as intended well done team! 1 Link to comment Share on other sites More sharing options...
rechigo 293 Posted May 21, 2021 Author Share Posted May 21, 2021 @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 Link to comment Share on other sites More sharing options...
BillOatman 492 Posted May 21, 2021 Share Posted May 21, 2021 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 Link to comment Share on other sites More sharing options...
pir8radio 1289 Posted May 21, 2021 Share Posted May 21, 2021 (edited) 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 Link to comment Share on other sites More sharing options...
kelly.clubb 8 Posted May 21, 2021 Share Posted May 21, 2021 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. Link to comment Share on other sites More sharing options...
grafxxx 4 Posted May 21, 2021 Share Posted May 21, 2021 (edited) 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 Link to comment Share on other sites More sharing options...
BaukeZwart 94 Posted May 21, 2021 Share Posted May 21, 2021 (edited) 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 Link to comment Share on other sites More sharing options...
grafxxx 4 Posted May 21, 2021 Share Posted May 21, 2021 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 1 Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now