-
Notifications
You must be signed in to change notification settings - Fork 53
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
Add automatic certificate generation for OneUI8 #65
base: main
Are you sure you want to change the base?
Conversation
For some reason, the image has python 3.6 and the library uses at least python |
I encountered the same errors. I don't know how to fix that. Maybe someone else has a answer to that |
…factor entrypoint.sh
I finally managed to get all things up and running. The last thing to do is to modify the Readme.md file accordingly to my changes. Maybe @Georift could help here. The new instrucitons to install the package autmatically to any Samsung TV is:
Example:
Wait until the server starts. The script will check every 5 seconds if the certificates are generated yet. Even if the script is not launched yet. So be patient, until there is a printed feedback from the script. Open: |
@Georift Nvm. I refactored the readme file. This pull request is ready for merging. |
@Georift Are you still maintaining this project? - Cause it looks like that you are very inactive. ^^ |
Hi. I pulled this down and built the docker image from the Dockerfile. I'm running the docker image on a headless server, which presents some problems. I can point my laptop browser at headless:7494/auth/start and I get to the first problem. It returns a JSON blob, instead of showing instructions. If I then visit the |
Actually this is the only thing where i can't actually help. As the generation script that i am using is from tizencertificates I experienced the same issue with the JSON blob. This occures because docker is not able to open a browser on the host machiene. Therefore you have to manually visit the side. A possible solution to this would be to directly return the Unfortunately i have to say that i am not responsible for you problem with your headless server, as the original script is not build to be run with a headless server. But we can come together and try to fix it together. |
Ah, no worries. I'd hoped you had a quick answer. I'm going to try and hack at it a bit and see if I can figure it out. |
If you find a solution i would be happy if you concider implmenting it into my PR |
Hey 👋 I just wanted to let you know that it worked fine directly from my laptop. I'll keep at it from the headless server, but I'm not optimistic. The Samsung page itself doesn't like non-full-browsers, so I don't think I'll get a curl solution (or similar) to work |
Running into an issue that I don't see documented in the Readme. Everything seems to work fine installing with the oneui8 steps but errors at the end with:
|
@Hippaduck
Are there any other log entrys that could be useful for debugging this issue? |
Jellyfin removed when the update happened and I can't find anywhere it may be installed still. The account is the same.
and the command I ran to kick it off
|
Hmm that is strange. I cannot identify any issues with the nstallation process here. Can you enter the container and check if the p12 certificates are in the right format? Get into the Containers Shell Check certs for expery date |
@Hippaduck @Epgenix When I got it to work, I used the email of my Samsung developer Account, not the email of the account signed into to the TV (which is my wife's). |
Interesting, i never had an Samsung Developer Account, i straightup used my normal Samsung Account. |
Okay, I've done a bit of research, and I haven't found an obvious way to do this headless. |
Yeah i already tried to make it as headless as possible for everyone. But i think this is the limit. There is a need for user feedback in form of a browser window with callback. |
And actually what i'm now wondering when will this PR get merged. Is it possible that @Georift is not longer maintaining this? |
I tried to use it but I get this error:
|
is install_python.sh in the directory present? |
Seems strange that it wouldn't error out on the earlier references to the same file. Sure it's not the hashbang /bin/bash from the top of the file that's not found? |
Hmm probably you right. Maybe the file format of install_python.sh is incorrect or something. I encountered a similar issue on the refactoring process of entrypoint.sh. I used dos2unix to remove the wierd hidden escape characters from windows and replace them with Unix ones. |
I put "#!/usr/bin/env bash" |
If so then you can directly push your change into my branch if you want to. I invited you into my forked repo. |
@Epgenix Dumb question, where do I get the device-id from? Is that the hostname of the TV? Might be worth putting in the readme |
Hmm seems to be the same issue as @Hippaduck had. Could you please check this Error and this one out. |
Ill rewrite th script oo that with |
The script improvements are ready. Let me know what you all think of this. Please feel free fo reach out if any errors occur. |
--get-device-id It gives me the model code of the TV. Is this correct?
I was able to complete the installation without any problems using the "Unique Device ID" value.
To avoid problems the correct value is: "Unique Device ID". |
Also if any issues occur check this out |
Originally posted by @Epgenix in #51
So i've been working in this for a few hours now (mostly figuring some things out)
This pull request is not tested yet, as i'm facing some issues with the building process of the Docker Image.
Docker is complaining that the version requested for fastapi is not existing. (It is)
I guess there is some work to do to get it up running.
But the idea is here, and maybe someone is picking up on this. I will definitely update this PR with some changes in the coming days.