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

Mutliple PLEX servers #9

Closed
djguu opened this issue Jan 2, 2021 · 31 comments
Closed

Mutliple PLEX servers #9

djguu opened this issue Jan 2, 2021 · 31 comments
Assignees
Labels
enhancement New feature or request

Comments

@djguu
Copy link

djguu commented Jan 2, 2021

Authentication failed : org.simpleframework.xml.core.PersistenceException: Element 'Server' is already used with @org.simpleframework.xml.Element(name=Server, type=class fr.arsenelapostolet.plexrichpresence.model.Server, data=false, required=false) on field 'Server' private fr.arsenelapostolet.plexrichpresence.model.Server fr.arsenelapostolet.plexrichpresence.model.MediaContainerServer.Server at line 4
@Ombrelin Ombrelin changed the title Unable to log in Mutliple PLEX servers Jan 2, 2021
@Ombrelin Ombrelin self-assigned this Jan 2, 2021
@Ombrelin Ombrelin added the bug Something isn't working label Jan 2, 2021
@Ombrelin
Copy link
Owner

Ombrelin commented Jan 2, 2021

Yes it's the multiple plex server error, I forgot about it. I'm going to look into it

@djguu
Copy link
Author

djguu commented Jan 4, 2021

Thank you @Ombrelin !

@Ombrelin Ombrelin added enhancement New feature or request and removed bug Something isn't working labels Jan 21, 2021
@Ombrelin
Copy link
Owner

The bug has been fixed, the first plex server is now selected. I'll improve that either by polling all the user's server or by featuring a server selection UI.

@djguu
Copy link
Author

djguu commented Jan 25, 2021

@Ombrelin I tried to login but it doesnt log me in
INFO - Authentication failed : Unable to connect to plex server.

@Ombrelin
Copy link
Owner

Mmmh I tested with multiple server myself, might be something else. We'll release a new version soon with a new login technique so maybe it'll fix this. Keep me posted if it doesn't !

@djguu
Copy link
Author

djguu commented Jan 25, 2021

I'll try to be on top when you release the new version :) Sometimes I skip the notifications, too many of them

@djguu
Copy link
Author

djguu commented Feb 1, 2021

@Ombrelin it's working now :D
Slight issue (not for me), your app uses 300mb+ of ram, I can see a problem with preople with less then 16gb of ram

@djguu
Copy link
Author

djguu commented Feb 1, 2021

Oh and, if I'm not on the plex app or the web interface, it says that I'm idle on discord, is that how it's supposed to be? Not really sure how hard it is to detect that you are on plex app or not

@mclarence
Copy link
Contributor

Oh and, if I'm not on the plex app or the web interface, it says that I'm idle on discord, is that how it's supposed to be? Not really sure how hard it is to detect that you are on plex app or not

I just implemented this.

@Ombrelin let me know if you want to submit PR for this.

@Ombrelin
Copy link
Owner

Ombrelin commented Feb 1, 2021

Indeed that would be very helpful !

@Ombrelin
Copy link
Owner

Ombrelin commented Feb 1, 2021

@Ombrelin it's working now :D
Slight issue (not for me), your app uses 300mb+ of ram, I can see a problem with preople with less then 16gb of ram

Glad to hear the new login fixed your problem !
For the memory usage, I'll look into it

@djguu
Copy link
Author

djguu commented Feb 1, 2021

Thank you guys @Ombrelin and @mclarence ❤️
I would like to help with this, but I don't have enough knowledge of Java, specifically never used a Java framework like spring boot

@Ombrelin
Copy link
Owner

Ombrelin commented Feb 1, 2021

No problem ! Any input is valuable 😉
But don't be afraid of the Spring Boot usage in this project, it's only leveraged for Dependency Injection nothing too fancy

@djguu
Copy link
Author

djguu commented Feb 2, 2021

@Ombrelin unfortunately, I can't get it to work now, it doesn't capture that I am watching something

@Ombrelin
Copy link
Owner

Ombrelin commented Feb 2, 2021

It says no session detected whereas you have a session playing on PLEX ?

@djguu
Copy link
Author

djguu commented Feb 2, 2021

yup..
image

@mclarence
Copy link
Contributor

You have multiple plex servers right?

Are you playing the media (shown in the screenshot) from another server or your main one?

Does it work on your other server?

@djguu
Copy link
Author

djguu commented Feb 3, 2021

tried both, neither one of them showed up in here

@djguu
Copy link
Author

djguu commented Feb 3, 2021

Before it worked fine, on 1.3

@Ombrelin
Copy link
Owner

Ombrelin commented Feb 3, 2021

It's very weird, since the server fetching process did not change between 1.3 et 1.4, @mclarence correct me if I'm mistaken

@mclarence
Copy link
Contributor

No, it has not changed.

I will probably post a new jar file of the app with increased verbosity in the logging so that we can pin point the cause of the issue.

@mclarence
Copy link
Contributor

plex-rich-presence-1.4.zip

Please run the above file with java -jar plex-rich-presence.jar -Dlogging.level.root=DEBUG and post the output from the command line here.

@djguu
Copy link
Author

djguu commented Feb 15, 2021

Sorry I haven't found the time to do it yet, Noticed there was a similar issue with the other user, so I guess it doesn't have a fix?

@mclarence
Copy link
Contributor

Yes, if you are not the owner of the plex server then the program will not detect the streams.

However, just post the debug log on pastebin and i'll have a look at it.

@mclarence
Copy link
Contributor

@Ombrelin it's working now :D
Slight issue (not for me), your app uses 300mb+ of ram, I can see a problem with preople with less then 16gb of ram

Glad to hear the new login fixed your problem !
For the memory usage, I'll look into it

In regards to memory usage, i've tested it with AdoptOpenJDK 11 + OpenJ9 JVM and it seems to have reduced the usage by quite a bit. From around 250-300 MB to around 160-180MB.

@Ombrelin
Copy link
Owner

That's good news ! I guess you added JavaFX as a Maven dependancy then ?

@mclarence
Copy link
Contributor

Yes.

jmetro had to be bumped up to 11.6.14 also.

@Ombrelin
Copy link
Owner

Okay ! My only fear with bumping the Java version is that people most commonly have JRE 8 installed by default.

@mclarence
Copy link
Contributor

Maybe you can bundle it with the windows installer.

@Ombrelin
Copy link
Owner

Yes I think there is definitely an option in launch4j to do that

@Ombrelin
Copy link
Owner

Ombrelin commented Oct 8, 2022

Should be fixed in 2.0 version !

@Ombrelin Ombrelin closed this as completed Oct 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants