search

osu-capstone-cs72 / cs-applied-plan-portal

A portal that streamlines the planning process for OSU CS Applied students and advisors
https://applied-plan-portal.herokuapp.com
MIT License
2 stars 5 forks source link

Expired login tokens should redirect to ONID login #76

Open JacksGo opened 4 years ago

JacksGo commented 4 years ago

I just ran into this about five minutes ago while finishing up the refactor: If you keep a session open until the login expires, you'll eventually try to navigate to a page and receive a 404 error. This isn't because the page has been deleted, but rather because the user has effectively logged out, and the API seems to default to responding to unauthorized requests with 404 instead of 403. Especially in the context of a plan page, this is extremely nerve wracking and should be fixed by redirecting unauthorized page requests to the ONID login.

silverware13 commented 4 years ago

Sure. I will look into this.

JacksGo commented 4 years ago

On hold. It's possible the server died on me, because I now can't start my local build with the same missing cookie error as before.