Skip to content

Commit c0ecce0

Browse files
author
Stephane Landelle
committed
Merge pull request AsyncHttpClient#504 from haed/master
OAuth signature calculator with userAuth#key check
2 parents db40594 + ba9723b commit c0ecce0

File tree

3 files changed

+8
-3
lines changed

3 files changed

+8
-3
lines changed

api/src/main/java/org/asynchttpclient/date/RFC2616DateParser.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ private Tokens tokenize() {
8383
}
8484

8585
// finish lastToken
86-
if (inToken = true)
86+
if (inToken == true)
8787
ends[tokenCount++] = end;
8888

8989
return new Tokens(starts, ends, tokenCount);

api/src/main/java/org/asynchttpclient/oauth/OAuthSignatureCalculator.java

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,9 @@ public String calculateSignature(String method, String baseURL, long oauthTimest
126126
allParameters.add(KEY_OAUTH_NONCE, nonce);
127127
allParameters.add(KEY_OAUTH_SIGNATURE_METHOD, OAUTH_SIGNATURE_METHOD);
128128
allParameters.add(KEY_OAUTH_TIMESTAMP, String.valueOf(oauthTimestamp));
129-
allParameters.add(KEY_OAUTH_TOKEN, userAuth.getKey());
129+
if (userAuth.getKey() != null) {
130+
allParameters.add(KEY_OAUTH_TOKEN, userAuth.getKey());
131+
}
130132
allParameters.add(KEY_OAUTH_VERSION, OAUTH_VERSION_1_0);
131133

132134
if (formParams != null) {
@@ -164,7 +166,9 @@ public String constructAuthHeader(String signature, String nonce, long oauthTime
164166
StringBuilder sb = new StringBuilder(200);
165167
sb.append("OAuth ");
166168
sb.append(KEY_OAUTH_CONSUMER_KEY).append("=\"").append(consumerAuth.getKey()).append("\", ");
167-
sb.append(KEY_OAUTH_TOKEN).append("=\"").append(userAuth.getKey()).append("\", ");
169+
if (userAuth.getKey() != null) {
170+
sb.append(KEY_OAUTH_TOKEN).append("=\"").append(userAuth.getKey()).append("\", ");
171+
}
168172
sb.append(KEY_OAUTH_SIGNATURE_METHOD).append("=\"").append(OAUTH_SIGNATURE_METHOD).append("\", ");
169173

170174
// careful: base64 has chars that need URL encoding:

api/src/main/java/org/asynchttpclient/util/ProxyUtils.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -207,6 +207,7 @@ public ProxyServer select(URI uri) {
207207
case HTTP:
208208
if (!(proxy.address() instanceof InetSocketAddress)) {
209209
log.warn("Don't know how to connect to address " + proxy.address());
210+
return null;
210211
} else {
211212
InetSocketAddress address = (InetSocketAddress) proxy.address();
212213
return new ProxyServer(Protocol.HTTP, address.getHostName(), address.getPort());

0 commit comments

Comments
 (0)