battlecode / galaxy

MIT License
10 stars 3 forks source link

Routing Improvements #797

Closed lowtorola closed 4 months ago

lowtorola commented 5 months ago

Fixes some routing insecurities and improves flow of control around episode switching.

Closes #794 :rocket:

acrantel commented 4 months ago

just to clarify - does "redirect to 404 not found page" mean that we should navigate to something like /notfound or does it mean that we will display the "page not found" message on the screen? currently it seems to be the "page not found" message.

(if the current behavior is the expected behavior, then this PR lgtm!)

lowtorola commented 4 months ago

@acrantel Yes it navigates to the "Page Not Found" message screen. I thought that seemed logical since technically for those bad URLs they are trying to access a page that doesn't exist

lowtorola commented 4 months ago

:shipit: