search

llaske / sugarizer-server

Sugarizer Server allow deployment of Sugarizer on a local server, for example on a school server.
Apache License 2.0
42 stars 92 forks source link

Sugarizer-Server LogOut Bug #328

Closed MananJain2002 closed 2 years ago

MananJain2002 commented 2 years ago

https://user-images.githubusercontent.com/63538502/163693625-64a86105-0caa-4617-bd9f-b9414d0c39ec.mp4

After logging off and doing undo it should redirected towards login page

llaske commented 2 years ago

It's not bug. Clicking back button could always cause errors.

MananJain2002 commented 2 years ago

Hi @llaske sir I'm not sure why it's not a bug even if it's a children's app its better for a website to be more securer anyway I managed to fix this issue here https://github.com/llaske/sugarizer-server/pull/329 please do review it. I have even attached a video for demonstration.

MananJain2002 commented 2 years ago

Hi mistakenly I made a pull request to master branch now I have changed it to dev branch here #330 please review it.

llaske commented 2 years ago

Not reproducible on recent Sugarizer Server version.