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

Modified response code 400 to 401 #962

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Modified response code 400 to 401 #962

wants to merge 2 commits into from

Commits on Jun 4, 2019

  1. Modified response code 400 to 401 

    When passing invalid client credentials (either client_id or client_secret) when requesting an access token, instead of returning a 400 response code, it should be returning a 401 response code for unauthorized. Limited it to the case where all conditions (public/non public) are passed and its only about the credentials. TokenControllerTest modified accordingly.
    @whvandervelde
    whvandervelde committed Jun 4, 2019
    Configuration menu
    Copy the full SHA
    8f1570b View commit details
    Browse the repository at this point in the history

  2. Added missing test 

    - added missing test for client credentials and the 400 -to 401 response code change
    @whvandervelde
    whvandervelde committed Jun 4, 2019
    Configuration menu
    Copy the full SHA
    b94f72b View commit details
    Browse the repository at this point in the history