cedar-policy / cedar-java

Java bindings for the Cedar language
https://www.cedarpolicy.com
Apache License 2.0
42 stars 19 forks source link

Update BasicAuthorizationEngine.java to fix logger error #92

Closed ChadAtAlbanero closed 8 months ago

ChadAtAlbanero commented 8 months ago

No matching issue

Removed superfluous log statement template placeholder that causes an IllegalArgumentException from BasicAuthorizationEngine::call. It appears that the full request log was moved to the subsequent LOG.trace statement, but the corresponding placeholder in this line was not removed. This can cause authorization requests to fail if the log level is debug or lower. This fix is the same as PR #90 , but targets the main branch instead of the current MavenCentral release branch.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.