Fix authentication type names broken by 0b5441a8
0b5441a8
introduced an enum for authentication types and changed the
case of digest and basic authentication type names to all uppercase.
This broke digest authentication at least when using Gerrit as the git
server.
According to RFC2617 [1] "Basic" and "Digest" is the literal to be used
in authentication headers and not "BASIC" [1] and "DIGEST" [2].
According to RFC4559 "Negotiate" [3] is used for SPNEGO based
authentication.
[1] http://tools.ietf.org/html/rfc2617#page-5
[2] http://tools.ietf.org/html/rfc2617#page-8
[3] http://tools.ietf.org/html/rfc4559#page-3
Bug: 435866
Change-Id: I6173aff9352d7def225cafe2d73e5176ad40dff0
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
This commit is contained in:
parent
c1f11fafce
commit
f80b41fbc1
|
@ -85,24 +85,44 @@ public enum Type {
|
|||
public HttpAuthMethod method(String hdr) {
|
||||
return None.INSTANCE;
|
||||
}
|
||||
|
||||
@Override
|
||||
public String getSchemeName() {
|
||||
return "None"; //$NON-NLS-1$
|
||||
}
|
||||
},
|
||||
BASIC {
|
||||
@Override
|
||||
public HttpAuthMethod method(String hdr) {
|
||||
return new Basic();
|
||||
}
|
||||
|
||||
@Override
|
||||
public String getSchemeName() {
|
||||
return "Basic"; //$NON-NLS-1$
|
||||
}
|
||||
},
|
||||
DIGEST {
|
||||
@Override
|
||||
public HttpAuthMethod method(String hdr) {
|
||||
return new Digest(hdr);
|
||||
}
|
||||
|
||||
@Override
|
||||
public String getSchemeName() {
|
||||
return "Digest"; //$NON-NLS-1$
|
||||
}
|
||||
},
|
||||
NEGOTIATE {
|
||||
@Override
|
||||
public HttpAuthMethod method(String hdr) {
|
||||
return new Negotiate(hdr);
|
||||
}
|
||||
|
||||
@Override
|
||||
public String getSchemeName() {
|
||||
return "Negotiate"; //$NON-NLS-1$
|
||||
}
|
||||
};
|
||||
/**
|
||||
* Creates a HttpAuthMethod instance configured with the provided HTTP
|
||||
|
@ -112,6 +132,13 @@ public HttpAuthMethod method(String hdr) {
|
|||
* @return a configured HttpAuthMethod instance
|
||||
*/
|
||||
public abstract HttpAuthMethod method(String hdr);
|
||||
|
||||
/**
|
||||
* @return the name of the authentication scheme in the form to be used
|
||||
* in HTTP authentication headers as specified in RFC2617 and
|
||||
* RFC4559
|
||||
*/
|
||||
public abstract String getSchemeName();
|
||||
}
|
||||
|
||||
static final String EMPTY_STRING = ""; //$NON-NLS-1$
|
||||
|
@ -270,7 +297,7 @@ void authorize(final String username, final String password) {
|
|||
void configureRequest(final HttpConnection conn) throws IOException {
|
||||
String ident = user + ":" + pass; //$NON-NLS-1$
|
||||
String enc = Base64.encodeBytes(ident.getBytes("UTF-8")); //$NON-NLS-1$
|
||||
conn.setRequestProperty(HDR_AUTHORIZATION, type.name()
|
||||
conn.setRequestProperty(HDR_AUTHORIZATION, type.getSchemeName()
|
||||
+ " " + enc); //$NON-NLS-1$
|
||||
}
|
||||
}
|
||||
|
@ -357,7 +384,7 @@ void configureRequest(final HttpConnection conn) throws IOException {
|
|||
v.append(e.getValue());
|
||||
v.append('"');
|
||||
}
|
||||
conn.setRequestProperty(HDR_AUTHORIZATION, type.name()
|
||||
conn.setRequestProperty(HDR_AUTHORIZATION, type.getSchemeName()
|
||||
+ " " + v); //$NON-NLS-1$
|
||||
}
|
||||
|
||||
|
@ -514,7 +541,7 @@ void configureRequest(HttpConnection conn) throws IOException {
|
|||
byte[] token = context.initSecContext(prevToken, 0,
|
||||
prevToken.length);
|
||||
|
||||
conn.setRequestProperty(HDR_AUTHORIZATION, getType().name()
|
||||
conn.setRequestProperty(HDR_AUTHORIZATION, getType().getSchemeName()
|
||||
+ " " + Base64.encodeBytes(token)); //$NON-NLS-1$
|
||||
} catch (GSSException e) {
|
||||
IOException ioe = new IOException();
|
||||
|
|
Loading…
Reference in New Issue