lmccay opened a new pull request, #1038: URL: https://github.com/apache/knox/pull/1038
(It is very **important** that you created an Apache Knox JIRA for this change and that the PR title/commit message includes the Apache Knox JIRA ID!) ## What changes were proposed in this pull request? Currently, the support for client_id and client_secret treats the inclusion of the CLIENT_ID as a formality of the client credentials flow and since the actual client_id is resolvable from the client_secret, it is ignored. While there it may be arguable whether we need to enforce this, it seems a reasonable expectation that they should match. Let's close that gap. We may need to decide whether we want to make that configurable. Is there a feature hidden in there somewhere? ## How was this patch tested? Ran existing unit tests and added new tests. Tested manually. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@knox.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
