search

simplerisk / code

The currently released SimpleRisk source code.
https://www.simplerisk.com
93 stars 51 forks source link

reset_password.php references unavailable css #49

Open mmickelson opened 4 weeks ago

mmickelson commented 4 weeks ago

After installing Simple Risk core, I found that the reset password screen was taken over by the Simple Risk logo. Shrinking the width of the page make the form usable.

clipboard-202409101843-6za12

After investigating, it seems that the css files were 404.

jsokol commented 4 weeks ago

@mmickelson Thanks for letting us know. Can you please let us know what version of SimpleRisk you are running and what installation method was used? We have a new release ready to launch on Friday and I don't see this issue there. It's also a bit perplexing as these CSS files you are referencing shouldn't even be under that "css" directory any longer. I'm guessing this is something we found and fixed for this new release.

mmickelson commented 4 weeks ago

The version is 20240819-001. I'm asking IT which specific method they used to install it.

jsokol commented 4 weeks ago

Did you recently upgrade to this version? Best I can tell, this is not an issue with that release of SimpleRisk as I am unable to replicate on an instance running the same version. If I had to guess, it sounds like maybe your browser cached that page and is trying to load CSS that no longer is relevant. Try hitting it in Incognito Mode or in another browser to see if you continue to get the same results.

mmickelson commented 4 weeks ago

It's a fresh install completed yesterday. We haven't used the software before, I just learned about it last week and asked for an instance to install it on. I provided the link to the download page to IT so I am sure it was freshly downloaded.

The IT rep was able to reproduce the issue and provided the screenshot I added. I see in the reset_password.php file, around line 84, it references these files in the CSS folder. https://github.com/simplerisk/code/blob/master/simplerisk/reset_password.php#L84

mmickelson commented 4 weeks ago

IT used the docker-compose version found here: https://github.com/simplerisk/docker/blob/master/stack.yml

jsokol commented 4 weeks ago

OK, that's helpful. I think I may see where the issue lies. Let me feed this off to our development team to have a look at it.