getnamo / BLUI-Unreal

CEF based HTML UI for the Unreal Engine.
MIT License
350 stars 73 forks source link

Cookies are automatically deleted due to CEF bug. How to update plugin with newer CEF version? #60

Closed vle07 closed 1 year ago

vle07 commented 3 years ago

I think the CEF version used for this plugin has a bug that automatically deletes cookies, so cookies can't persist between sessions: https://bitbucket.org/chromiumembedded/cef/issues/2890/all-the-cookies-are-deleted-automatically

Are there any plans to update the CEF version? I can also try updating it myself, but it seems really involved. So far, I've download a recent CEF build from: https://cef-builds.spotifycdn.com/index.html. Then I've replaced the cefsimple test project with the blubrowser project here: https://github.com/getnamo/blubrowser. Using CMake, I generate the project files. I'm a little unclear about the /MT or /MD outputs. Do I need to build the solution twice, once with each configuration to generate all of the necessary files? If you have any tips or resources for updating the code to get the solution to build (and what to do after that), it'd be appreciated. This is a great plugin and is much easier to extend than the default UE4 web browser. Thanks!

Edit: Seems like cookies work in a packaged project when setting "persist_session_cookies" and "persist_user_preferences" to true. Not sure why it doesn't work when using the UE4 editor.

getnamo commented 3 years ago

Interesting bug. I do occasionally re-sync CEF builds, but it won't be more often than ~ once every 6months. If you need an update sooner, https://github.com/getnamo/blubrowser is the correct repo to do it. You do need to change some settings in the .sln manually the /MT checkbox can be found under blu_ue4_process project Configuration Properties-> C/C++->Code Generation-> Runtime Library

I usually do a build all and then copy the required libs in Win/lib and all the updated process under Win/shipping. Essentially ensure Plugins\BLUI\ThirdParty\cef\Win is up to date with newest cef build and update your headers and it should work.

getnamo commented 1 year ago

Closing in favor of #81