LemonLDAP::NG: Remove getBearerSignature() override Bearer Authorization Tokens appear to be supported by v2.0: https://lemonldap-ng.org/documentation/2.0/oauth2handler. Change-Id: I58d55d9e00f7881ad7143214d929df0327268011
diff --git a/src/main/java/com/googlesource/gerrit/plugins/oauth/LemonLDAPApi.java b/src/main/java/com/googlesource/gerrit/plugins/oauth/LemonLDAPApi.java index 7b66c25..55ebf38 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/oauth/LemonLDAPApi.java +++ b/src/main/java/com/googlesource/gerrit/plugins/oauth/LemonLDAPApi.java
@@ -15,8 +15,6 @@ package com.googlesource.gerrit.plugins.oauth; import com.github.scribejava.core.builder.api.DefaultApi20; -import com.github.scribejava.core.oauth2.bearersignature.BearerSignature; -import com.github.scribejava.core.oauth2.bearersignature.BearerSignatureURIQueryParameter; import com.github.scribejava.core.oauth2.clientauthentication.ClientAuthentication; import com.github.scribejava.core.oauth2.clientauthentication.RequestBodyAuthenticationScheme; @@ -45,11 +43,4 @@ public ClientAuthentication getClientAuthentication() { return RequestBodyAuthenticationScheme.instance(); } - - // TODO(davido): Remove this override, if BearerSignatureAuthorization - // request header field is supported. - @Override - public BearerSignature getBearerSignature() { - return BearerSignatureURIQueryParameter.instance(); - } }