Skip to content
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

Plex Friend not being imported #4989

Closed
Arnauec opened this issue Jul 25, 2023 · 15 comments
Closed

Plex Friend not being imported #4989

Arnauec opened this issue Jul 25, 2023 · 15 comments

Comments

@Arnauec
Copy link

Arnauec commented Jul 25, 2023

Summary

I have two Friends on Plex, I'm able to successfully import one (and if I delete it it gets re-imported as expected), but the other one won't get imported.

The only difference I can see in them is that one has a Name and an username (failed import), and the other only has a username. But I doubt this is it.

image image

Any configuration or permission on my friend side that I should be aware of? When I trigger the User Import it runs successfully and no logs are generated.

Also worth mentioning, the friend that isn't syncing has its own NAS and we are sharing them between us. The friend that was successfully imported only has access to my shared folders.

Thank you!

Ombi Version

4.39.1

What platform(s) does this occur on?

Docker

What database are you using?

SQLite (Default)

Relevant log output

When I run the User Import there are no logs being generated related with the import.
@tidusjar
Copy link
Member

That is odd.

I would have expected some logs when running the importer to tell you why it didn’t import.

Can you run the importer manually and check the logs?

@Arnauec
Copy link
Author

Arnauec commented Jul 25, 2023

There are no logs at all generated when I run the User Importer.

This is the last entry of the logs before and after I run the importer, just for reference now it is 15:05. This is unrelated with the import and no extra logs are generated.

2023-07-25 13:37:51.522 +02:00 [ERR] Could not test Plex
System.Net.Http.HttpRequestException: Connection refused (192.168.0.231:17878)
---> System.Net.Sockets.SocketException (111): Connection refused
at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.ThrowException(SocketError error, CancellationToken cancellationToken)
[...]

I've also intercepted the request I'm doing with a proxy:

POST /ombi/api/v1/Job/plexUserImporter/ HTTP/2
Host: ombi.estebanell.com
Cookie:
Content-Length: 187
Sec-Ch-Ua: "Not/A)Brand";v="99", "Google Chrome";v="115", "Chromium";v="115"
Accept: application/json, text/plain, /
Content-Type: application/json
Sec-Ch-Ua-Mobile: ?0
Authorization:
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/115.0.0.0 Safari/537.36
Sec-Ch-Ua-Platform: "macOS"
Origin: https://ombi.estebanell.com
Sec-Fetch-Site: same-origin
Sec-Fetch-Mode: cors
Sec-Fetch-Dest: empty
Referer: https://ombi.estebanell.com/ombi/Settings/UserManagement
Accept-Encoding: gzip, deflate
Accept-Language: en-ES,en;q=0.9,ca-ES;q=0.8,ca;q=0.7,es-ES;q=0.6,es;q=0.5,en-GB;q=0.4,en-US;q=0.3

{"headers":{"normalizedNames":{},"lazyUpdate":[{"name":"Content-Type","value":"application/json","op":"s"}],"headers":{},"lazyInit":{"normalizedNames":{},"lazyUpdate":null,"headers":{}}}}

And what I get in response is:

HTTP/2 200 OK
Server: nginx/1.25.1
Date: Tue, 25 Jul 2023 13:02:58 GMT
Content-Type: application/json; charset=utf-8
Content-Length: 4
Access-Control-Allow-Credentials: true
Access-Control-Allow-Origin: https://ombi.estebanell.com
Vary: Origin
Strict-Transport-Security: max-age=31536000

true

@paris-ci
Copy link

I have the same problem. No logs too

@vghoost360
Copy link

same problem here have it for a few days now reinstalled everything on docker even have used new configs for plex and ombi but it will not import new registerd users that have signed up the last 3 months or so checked users in yml no difference with other users only thing making them differend is there sign up date

@tidusjar
Copy link
Member

tidusjar commented Jul 26, 2023 via email

@tidusjar
Copy link
Member

Can someone send me the output from this call either via email tidusjar@gmail.com or on Discord

https://plex.tv/api/users?X-Plex-Token=PLEX_TOKEN

You can find your PLEX_TOKEN here: https://support.plex.tv/articles/204059436-finding-an-authentication-token-x-plex-token/

@TheDarkOne96
Copy link

I have the same issue, no logs are generated when the importer function is called.
I can still see the missing users using this link: https://plex.tv/api/users?X-Plex-Token=PLEX_TOKEN

@klepp0906
Copy link

klepp0906 commented Jul 27, 2023

same issue. super weird. wife was in my ombi until some seemingly random point a day or two ago. only things i recall changing in the last few days was removing some items from her watchlist on plex. (watchlist request is set up)

now she's not there and no matter how many times or ways i try to run the importer she isnt coming back lol.

everyone else seems untouched which leads me to believe it has to have something to do with having removed those watchlist items from my tvs plex client.

already tried removing/re-adding the server thinking maybe some kinda token thing but it found the same server, same token, no bueno.

have a few errors in my logs but unsure which if any are relevant. gonna pastebin em momentarily.

so there was a whole lot of this

https://privatebin.net/?477f25e8cdf5c5b4#HFRvaiY2DmwfdisNLqPQ1vsM7xkNoLwG2HZjDWMwAEjP

and this

https://privatebin.net/?8deb8f1b64b6fd47#9W8xVUgAxGmkEFHuVge8HBbVxmCQb83x2u61z8TXukFV

followed by these errors

  1. https://privatebin.net/?69a50b71bb5150d4#7mENyTVXz6qBsCjgk9ZYyssoPMTGmii2KwfYV3dju6Gm

  2. https://privatebin.net/?5b45133605fcce00#D3CuhLwiZSEBN3daNwPipN1cZuKSL2eh5k8Kn7XwXHaB

  3. https://privatebin.net/?a5fdcbacf99a676a#FVMAMvY4PFrrQJ7bViXrmJBfEhWwvLnVgZw8HmH3Jjde

with any luck something there will be helpful. also worth mentioning I ran that call and it listed everyone in my plex. 3 people are not imported into ombi. All 3 are my family members. my 2 kids which are managed users so they were already not being imported by design. my wife however we created a plex account for her intentionally to replace the managed one she had. she was imported fine until... she wasnt :p

am on windows
4.43.2 being started ala scheduled task. updated a few times in a few days though as releases were made available. at one point to address the admin user being purged bug, then i believe there was another release after that before the current. she could have poofed at any point in there, i wasnt checking unfortunately.

looking at old database backups apparently it happened at 12:00AM on the 21st. unsure if that coincided with 4.43.2 perhaps. maybe i ought to try a roll back and re import. I'm waaaaay too stupid to edit her back in manually without probably blowing the db to bits im sure.

UPDATE: so i downgraded to 4.43.1 and the importer worked fine, brought her back. its something introduced in 4.43.2. disabled plex user cleanup but afraid to roll back up in the meantime just in case it has zero to do with that setting.

@tidusjar
Copy link
Member

That’s helpful, thank you.

@tidusjar
Copy link
Member

Can someone try the new build?

@Arnauec
Copy link
Author

Arnauec commented Jul 28, 2023

Hey, just tried the new build and the problem is still there. Also still no logs.

image

@tidusjar
Copy link
Member

@klepp0906 are you able to send me the output from the API call?

@klepp0906
Copy link

klepp0906 commented Jul 28, 2023

sure, sent it over in disc

@tidusjar
Copy link
Member

Ok, found the issue.

The Plex API can report users as "Home" users if they were a pure home user and then converted to a non-managed account.
I recently added a check to not add home users by checking the HomeUser attribute from the API, thinking that it would only be true for pure home users.

I've now removed that check and it will all work as it previously did.

@Arnauec
Copy link
Author

Arnauec commented Jul 29, 2023

Hey @tidusjar, I can confirm that it works, I'll close the issue. Thank you very much!

@Arnauec Arnauec closed this as completed Jul 29, 2023
tidusjar added a commit that referenced this issue Jul 22, 2024
Co-authored-by: tidusjar <tidusjar@gmail.com>
Co-authored-by: sephrat <34862846+sephrat@users.noreply.github.com>
Co-authored-by: Conventional Changelog Action <conventional.changelog.action@github.com>
Co-authored-by: Teifun2 <Teifun2@users.noreply.github.com>
Co-authored-by: contrib-readme-bot <contrib-readme-action@noreply.com>
Co-authored-by: dr3amer <91037083+dr3am37@users.noreply.github.com>
Co-authored-by: echel0n <echel0n@sickrage.ca>
Co-authored-by: Marley <55280588+marleypowell@users.noreply.github.com>
Co-authored-by: Igor Borges <igor@borges.dev>
Co-authored-by: Lucane <Lucane@users.noreply.github.com>
Co-authored-by: mkgeeky <github@mkgeeky.xyz>
Co-authored-by: Miguel A Vico Moya <mvicomoya@gmail.com>
Co-authored-by: Hadrien <26697460+ketsapiwiq@users.noreply.github.com>
Co-authored-by: Victor Usoltsev <bernarden@users.noreply.github.com>
Co-authored-by: Wesley King <kingwe92@gmail.com>
Co-authored-by: Lea <me@janderedev.xyz>
Co-authored-by: snyk-bot <snyk-bot@snyk.io>
Co-authored-by: snyk-bot <snyk-bot@snyk.io> [skip ci]
Co-authored-by: Jamie <tidusjar@gmail.com>
Co-authored-by: Drew <drewm727@hotmail.com> [skip ci]
Co-authored-by: ryan-c44 <54028283+ryan-c44@users.noreply.github.com>
Co-authored-by: Alexander Russell <ajex94@gmail.com>
Co-authored-by: Grygon <647846+Grygon@users.noreply.github.com>
Co-authored-by: phildups7 <60622768+phildups7@users.noreply.github.com>
Co-authored-by: Avi <357984+Unimatrix0@users.noreply.github.com>
Co-authored-by: Antonin <82907030+Antonin-Bruzard@users.noreply.github.com>
Co-authored-by: Drew <mcminn.drew@gmail.com>
Co-authored-by: Joel Samson <joel.samson@hotmail.com>
Co-authored-by: David Benson <dben@users.noreply.github.com>
fix: Consistently reset loading flag when requesting movies on discover page. (#4777)
fix(sonarr): 🐛 Fixed an issue where the language list didn't correctly load for power users in the advanced options #4782
fix: Only log error messages from Microsoft (#4787)
fix(notifications): Fixed the Partially TV notifications going to the admin #4797 (#4799)
fix(translations): 🌐 New translations from Crowdin [skip ci] (#4801)
fix(sonarr): 🐛 Sonarr V4 should work now (#4810)
fixed (#4833)
fix(plex-watchlist): Lookup the ID from different sources when Plex doesn't contain the metadata (#4843)
fix(#4847): Invalid Discord request fixed, also fixed an issue where App Only users would not show as logged in on the user management page (#4848)
fix(discover): 🐛 Fixed the default poster not taking into account the base url in some scenarios #4845
fix(wizard): 🐛 Stop access to the wizard when you have already setup ombi (#4866)
fix(plex-oauth): 🐛 Fixed an issue where using OAuth you could log in as a Ombi Local user #4835
fix: Some minor tweaks to the movie info panel (#4883)
fix(sonarr): 🐛 Stop the sonarr version endpoint from breaking when Sonarr is down #4895
fix: Support duplicates in Emby/JF collections (#4902)
fix(discover): Fix denied requests displayed as approved (#4901)
fix: Fix denied movie shown as 'processing request' in details view (#4900)
fix(#4906): 🐛 Fixed an issue with power users and permissions
fix: Cron Validation (#4842)
fix(lidarr): Change monitor to Existing to properly add artist #3597
fix(API): Allow RequestOnBehalf rights if requested from the API (#4919)
Fixes #4610
fix: added media type tag to media type text (#4638)
fix(sickrage): Fixed issue with incorrect handling of SiCKRAGE episode results returned during episode status changes, now expects array of objects from data path if present (#4648)
fix: Missing Poster broken link fix (#4637)
fix: Improve Swagger documentation (#4652)
fix(API): Fix pagination in some edge cases (#4649)
fix(discover): Carousel touch not working when scrolling page and recommendations and similar movie navigation (#4633)
fix: 🐛 Fixed the Request on Behalf of having blanks (#4667)
fix(images): Retry images with a backoff when we get a Too Many requests from TheMovieDb #4685
fix: Landing and Login page improvements (#4690)
fix(discover): 🐛 Created new Image component to handle 429's from TMDB (#4698) and fixed #4635 (#4699)
fix: Override Sonarr V3 Profiles endpoint (#4678)
fix(4K) :4K request fixes (#4702)
fix(translations): 🌐 New translations from Crowdin [skip ci] (#4713)
fix: fixed trakt image not loading when base url present (#4711)
fix: 🐛 Fixed missing externals (#4712)
fix: Log Microsoft warnings to log file (#4723)
fix: Localize recently requested on discover page (#4729)
Fix: Ombi.Api.Lidarr: Remove unused fields from ArtistAdd (#4727)
fix: Fix conflicting property name for Swagger (#4733)
fix: fixed stats controller (#4742)
fix(webhook): Remove added trailing slash from webhook URL #4710
fix(sonarr): 🐛 Cleaned up and removed Sonarr v3 option, sonarr v3 is now the default. This allows us to get ready for the upcoming Sonarr v4 (#4764)
fix: Fixes default image for recently requested items. (#4767)
Fixes build warnings. (#4769)
fix: Reworked the version check (#4719) (#4781)
fix: Unable to Delete Jellyfin Server (#4705) (#4780)
fix: Partially Available prevents further TV requests (#4768) (#4779)
fix(emby): Fix Emby played sync running a full sync during recently added sync (#4932)
Fixes #4947
fix: Fix various styling issues (#4935)
fix(translations): 🌐 New translations from Crowdin [skip ci] (#4926)
fix: upgrade @microsoft/signalr from 6.0.11 to 6.0.16 (#4964) [skip ci]
fix: upgrade primeng from 15.0.0-rc.1 to 15.4.1 (#4962) [skip ci]
fix: src/Ombi.Notifications/Ombi.Notifications.csproj to reduce vulnerabilities (#4969) [skip ci]
fix: upgrade @fortawesome/fontawesome-free from 6.1.2 to 6.4.0 (#4965) [skip ci]
fix: upgrade multiple dependencies with Snyk (#4963) [skip ci]
fix flaky tests (#4970) [skip ci]
fix: More automation tests mainly around the Plex Settings page (#4821)
fix: upgrade cypress-real-events from 1.7.4 to 1.8.1 (#4968) [skip ci]
fix: Remove Angular TSLint (#4973)
fix: upgrade zone.js from 0.11.8 to 0.13.0 (#4975)
fix: upgrade jquery from 3.6.1 to 3.7.0 (#4974)
fix: upgrade multiple dependencies with Snyk (#4961)
fixes (#4978)
fix(user-importer): 🐛 Fixed an issue where the cleanup wouldn't delete users #4812
fix(user-importer): Do not delete the Plex Admin as part of the user Importer cleanup #4870 (#4981)
Fixes #4957
fix: Remove old trending source (#4987)
fix(plex-api): Switch over to the new API to avoid deprecation & save… (#4986)
fix: switch back to the old plex friends API #4989
fix(user-importer): Fixed not importing all correct users #4989
fix: upgrade cypress-real-events from 1.8.1 to 1.9.1 (#5000) [skip ci]
fix: upgrade @microsoft/signalr from 6.0.18 to 6.0.20 (#4999) [skip ci]
fix: upgrade @fortawesome/fontawesome-free from 6.4.0 to 6.4.2 (#5005) [skip ci]
fix: upgrade @types/jquery from 3.5.16 to 3.5.17 (#5011) [skip ci]
fix: src/Ombi/ClientApp/package.json & src/Ombi/ClientApp/yarn.lock to reduce vulnerabilities (#5010) [skip ci]
fix: upgrade @microsoft/signalr from 6.0.21 to 6.0.22 (#5020)
fix: upgrade @types/jquery from 3.5.18 to 3.5.19 (#5022)
fix: upgrade zone.js from 0.13.1 to 0.13.2 (#5019)
fix: upgrade multiple dependencies with Snyk (#5030)
fix: upgrade cypress-real-events from 1.10.0 to 1.10.1 (#5014)
fix: upgrade jquery from 3.7.0 to 3.7.1 (#5015)
Fix: Linkify logo in newsletter (#5036)
fix: upgrade @types/jquery from 3.5.22 to 3.5.23 (#5034) [skip ci]
fix: upgrade @microsoft/signalr from 6.0.22 to 6.0.23 (#5032) [skip ci]
Fix(mutliple servers): attempting to cache the Plex Content in server (#5018)
fix(radarr-4k): 🐛 Fixed an issue when using Radarr 4k with user set quality profiles. There are now user quality profiles for 4k profile #5025
fix: src/Ombi/ClientApp/package.json & src/Ombi/ClientApp/yarn.lock to reduce vulnerabilities (#5072) [skip ci]
fix: src/Ombi/Ombi.csproj to reduce vulnerabilities (#5066) [skip ci]
fix: upgrade @fortawesome/fontawesome-free from 6.4.2 to 6.5.0 (#5053)  [skip ci]
fix: upgrade @types/jquery from 3.5.27 to 3.5.28 (#5049)  [skip ci]
fix: src/Ombi/ClientApp/package.json & src/Ombi/ClientApp/yarn.lock to reduce vulnerabilities (#5040)  [skip ci]
fix: upgrade moment from 2.29.4 to 2.30.1 (#5075) [skip ci]
fix: upgrade multiple dependencies with Snyk (#5073) [skip ci]
fix: upgrade multiple dependencies with Snyk (#5104) [skip ci]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants