Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Don't use GET for log in or out #58

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Don't use GET for log in or out #58

wants to merge 1 commit into from

Commits on Feb 23, 2021

  1. Don't use GET for log in or out 

    GET should never be used for logging in or out of an API. 
* GET is cached
   * you can't be sure you'll actually get a new log in/out and not a cached response
  * user credentials could be cached in the browser history or intermediate servers (!)
    @nrktkt
    nrktkt committed Feb 23, 2021
    Configuration menu
    Copy the full SHA
    01fab39 View commit details
    Browse the repository at this point in the history