OpenFactorioServerManager / factorio-server-manager

A tool to help manage Factorio multiplayer servers including mods and save games.
MIT License
557 stars 133 forks source link

Server manager docker container displays a blank page since update #134

Closed danstis closed 5 years ago

danstis commented 5 years ago

After the latest release I am getting a blank page when opening the Server Manager URL. I am using the docker container version, and have tested with a new clean container instance.

This is what I see on the main page: image

This is logged in the console: image

smoopmeister commented 5 years ago

Confirming i'm also getting the same error on the windows release.

mroote commented 5 years ago

Hmm sorry about that I seem to be getting the same error. I checked the error in development mode and it appears to be an error due to routing? @knoxfighter maybe you have an idea? I haven't made a build in a bit so I'm not sure if it's something on my end.

knoxfighter commented 5 years ago

I currently am fixing this problem. Its not that easy to find and fix, will take a few more hours :) Its caused by updating the react-router-dom

knoxfighter commented 5 years ago

PR is open. The login-page has styling problems, but should work now :)

Najsr commented 5 years ago

@knoxfighter Could you please make a temporary build before PR is merged and new released published? Thanks.

knoxfighter commented 5 years ago

@Najsr I created a temporary release: https://github.com/knoxfighter/factorio-server-manager/releases/tag/0.8.1.rc1 I cannot update the release on the docker hub.

keckrich commented 5 years ago

The new release did not fix the white page issue for me.

knoxfighter commented 5 years ago

The Linux release i created is fixed for me. Be careful: I didn't updated the docker images!

tardisx commented 5 years ago

The new release did not fix the white page issue for me.

Agreed. Still white screen for me with that release.

If I use incognito mode, I do see a login screen. So presumably something from the cookie state is breaking things. I suspect clearing cookies will fix that.

Najsr commented 5 years ago

@knoxfighter I don't use linux docker and it works now. Thanks!

mroote commented 5 years ago

I've merged in @knoxfighter's fix and updated the docker container for 0.8.1. Thanks for confirming @Najsr!

knoxfighter commented 5 years ago

@mroote thanks for fixing it completly, for some reason, this error did not occure on my side :(