-
Couldn't load subscription status.
- Fork 3
feat: Transform Source, fix: OBSWebsocket reset #50
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
base: main
Are you sure you want to change the base?
Conversation
|
@Xanacas How about splitting this PR into two parts: Source Transformation and OBS Websocket settings? This way, we can avoid blocking some changes due to delayed review or testing of others. |
|
Sure, Just let me know, How to do this. 😅 and as i am already working on it. Are you okay with removing obsIni in total and replace it with an individual Plugin config? |
We can achieve this by separating the changes related to the OBS WebSocket into a new pull request and removing those changes from the current one. As a result, we will end up with two pull requests: this one will contain only the changes for Source Transformation, while the new one will include only the WebSocket changes. We are going to investigate the issue with the connection to the OBS WebSocket to identify all instances where it occurs. While some people experience this issue, others do not, and we need to understand why. Additionally, we need to conduct a deeper investigation to assess the priority and seriousness of the problem. |
|
@Oleksandr-Podranetskyy My suggestion is, we implement an individual setting.json for the plugin and get completely rid of the obsini stuff. I've already started with this. Just need a bit feedback, if it's okay if I go this way. |
The plugin interacts with the OBS INI file to:
What are your thoughts on this? |
|
@Oleksandr-Podranetskyy In addition to this, the password that is currently created includes lots of special chars - however if I look the password up in OBS, it is displaying the special chars as URI-encoded. Means it gets even harder to simultaneously use loupedeck with other tools that make use of obs websocket server. The changes I made in this PR towards the websocket thing are the following: I'd like to go a step further now:
Step two (I'm happy to then open another pr if needed:) )
|
|
@Xanacas |
|
@Xanacas The root cause of the issue with OBS WebSocket settings is that the settings are now stored elsewhere, not in the OBS INI file. To resolve this issue, it seems we simply need to read and write these settings from and to the new location. |
|
@Oleksandr-Podranetskyy |
We can find this information here - https://github.com/obsproject/obs-websocket/blob/eed8a49933786383d11f4868a4e5604a9ee303c6/src/Config.cpp |
|
@Oleksandr-Podranetskyy My assumption is, that the removal of the credentials from the obs ini is due to security reasons. Not storing the websocket credentials permanently in a plain config file seems to be a valid reason. However, therefore I doubt, that we can read it somewhere else easily. as mentioned earlier, if you're okay with my suggested path, I'll gonna implement it. |
|
I dont quite get why cant have a solution where we enter the password ourselfs on Loupedecks side. Its not like we change passwords every day, quite the opposite. All other programms that use the websocket have the password stored in some way, loupedeck is the only one that tries to automate the process and during that, screws it up. |
|
@Xanacas are we still waiting on an update for this? Noticed I'm still dealing with either Loupedeck or StreamerBot working. Just wondering if there is any sort update. Thanks |
|
Is this still being worked on or does anyone have any ideas how I can finally connect Loupdeck and Streamer.bot to OBS at the same time? |
|
Not only is still an issue, the OBS plugin now regulary uninstalls itself too.... |
|
@St0l3nID I've recently switched over to Meld, but remember having this issue on OBS too. Funny thing is shortly before I switched I could have sworn it was working again. So I booted up my OBS profile, streamer bot and opened loupedeck again, and it all connected. Loupedeck 6.2.1.160, OBS 31.0.3, Streamer bot 1.0.1. I did have to update the IP information for the websocket connection in Streamer bot, I had changed my router in the last month, but ya, Streamer bot and Loupedeck are both connected and commands work too. I'd just double check your settings in case something did change during an update or something. Maybe a firewall is getting in the way? Who knows. Good luck. |
|
I'm on the newest version of OBS, Loupdeck and the plugin. Loupedeck randomises the OBS websocket password everytime it connects to it, which it will only do once, if i have to restart obs, loupedeck needs to be restarted before it. And to get a connection reliably you have to have loupedeck open, open obs (password gets randomised) and then connect the other apps somehow (by either disalbing auth or giving them the new password). This, frankly, stupid behaviour is consistent and reproducable. |
|
@St0l3nID that doesn't sound quite right. Like I mentioned I haven't opened OBS for a bit, but when I did, Streamer Bot and Loupedeck both connected. I haven't changed the password for that setup in who knows how long, but it was still the same password. I just updated my IP because of the router replacement as I mentioned before. My first thought would be Loupedeck is somehow corrupted or maybe an update didn't apply the fix? I'd try uninstalling loupedeck first, after backing up your profiles of course. And then reinstall and see. But try to look through your appdata to make sure it's all cleared out. Just really weird I had the same issue, but it's fixed, yet it's still ongoing for you. |
With this Pull Request, there are two changes coming:
I'm really sorry for everything that I might made wrong, this is my very very first open source contribution and the first non-school c# project I've ever worked on 🙈🙏🥳