-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add 307 Temp Redirect endpoints for app-handled URLs #46
Comments
@rye I just rediscovered this issue… what do you think about it? I think that I think it still sounds good. |
I think that this makes sense. It'd be useful. |
Here's an initial list of URLs and redirect locations: (also, I'm using 307 instead of 302 so that the original method [POST/PATCH/etc] is preserved; https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/307)
I've left out the course data and SIS endpoints, since they'll need some more thought (they're used dynamically to build other request strings.) |
For instance, I don't want to proxy the Directory webview through the server, but we could make /v1/link/directory 307 to Carleton.edu/directory to handle changes on their end.
I think we should make a named 307 endpoint for each URL that the app handles on its own - that means CARLS would have links for TheHub, Stalkernet, Moodle, and the Balances parser, at least, while AAO would have Balances, Directory, Moodle, … the KSTO endpoint?
The links in the Help wouldn't need to be put on the server, since they're changeable remotely.
The text was updated successfully, but these errors were encountered: