Adds ClientBase.AuthorizeRequest overloads that accept WebHeaderCollection.

AArnott · AArnott · commit 0b5012fe114c · 2012-11-11T21:05:22.000-08:00
Fixes #119
diff --git a/src/DotNetOpenAuth.OAuth2.Client/OAuth2/ClientBase.cs b/src/DotNetOpenAuth.OAuth2.Client/OAuth2/ClientBase.cs
@@ -87,7 +87,20 @@ public static void AuthorizeRequest(HttpWebRequest request, string accessToken)
 			Requires.NotNull(request, "request");
 			Requires.NotNullOrEmpty(accessToken, "accessToken");
 
-			OAuthUtilities.AuthorizeWithBearerToken(request, accessToken);
+			AuthorizeRequest(request.Headers, accessToken);
+		}
+
+		/// <summary>
+		/// Adds the necessary HTTP Authorization header to an HTTP request for protected resources
+		/// so that the Service Provider will allow the request through.
+		/// </summary>
+		/// <param name="requestHeaders">The headers on the request for protected resources from the service provider.</param>
+		/// <param name="accessToken">The access token previously obtained from the Authorization Server.</param>
+		public static void AuthorizeRequest(WebHeaderCollection requestHeaders, string accessToken) {
+			Requires.NotNull(requestHeaders, "requestHeaders");
+			Requires.NotNullOrEmpty(accessToken, "accessToken");
+
+			OAuthUtilities.AuthorizeWithBearerToken(requestHeaders, accessToken);
 		}
 
 		/// <summary>
@@ -99,6 +112,19 @@ public static void AuthorizeRequest(HttpWebRequest request, string accessToken)
 		public void AuthorizeRequest(HttpWebRequest request, IAuthorizationState authorization) {
 			Requires.NotNull(request, "request");
 			Requires.NotNull(authorization, "authorization");
+
+			this.AuthorizeRequest(request.Headers, authorization);
+		}
+
+		/// <summary>
+		/// Adds the OAuth authorization token to an outgoing HTTP request, renewing a
+		/// (nearly) expired access token if necessary.
+		/// </summary>
+		/// <param name="requestHeaders">The headers on the request for protected resources from the service provider.</param>
+		/// <param name="authorization">The authorization for this request previously obtained via OAuth.</param>
+		public void AuthorizeRequest(WebHeaderCollection requestHeaders, IAuthorizationState authorization) {
+			Requires.NotNull(requestHeaders, "requestHeaders");
+			Requires.NotNull(authorization, "authorization");
 			Requires.True(!string.IsNullOrEmpty(authorization.AccessToken), "authorization");
 			ErrorUtilities.VerifyProtocol(!authorization.AccessTokenExpirationUtc.HasValue || authorization.AccessTokenExpirationUtc < DateTime.UtcNow || authorization.RefreshToken != null, ClientStrings.AuthorizationExpired);
 
@@ -107,7 +133,7 @@ public void AuthorizeRequest(HttpWebRequest request, IAuthorizationState authori
 				this.RefreshAuthorization(authorization);
 			}
 
-			AuthorizeRequest(request, authorization.AccessToken);
+			AuthorizeRequest(requestHeaders, authorization.AccessToken);
 		}
 
 #if CLR4
diff --git a/src/DotNetOpenAuth.OAuth2/OAuth2/OAuthUtilities.cs b/src/DotNetOpenAuth.OAuth2/OAuth2/OAuthUtilities.cs
@@ -136,7 +136,20 @@ internal static void AuthorizeWithBearerToken(this HttpWebRequest request, strin
 			Requires.NotNullOrEmpty(accessToken, "accessToken");
 			ErrorUtilities.VerifyProtocol(accessToken.All(ch => accessTokenAuthorizationHeaderAllowedCharacters.IndexOf(ch) >= 0), OAuthStrings.AccessTokenInvalidForHttpAuthorizationHeader);
 
-			request.Headers[HttpRequestHeader.Authorization] = string.Format(
+			AuthorizeWithBearerToken(request.Headers, accessToken);
+		}
+
+		/// <summary>
+		/// Authorizes an HTTP request using an OAuth 2.0 access token in an HTTP Authorization header.
+		/// </summary>
+		/// <param name="requestHeaders">The headers on the request for protected resources from the service provider.</param>
+		/// <param name="accessToken">The access token previously obtained from the Authorization Server.</param>
+		internal static void AuthorizeWithBearerToken(WebHeaderCollection requestHeaders, string accessToken) {
+			Requires.NotNull(requestHeaders, "requestHeaders");
+			Requires.NotNullOrEmpty(accessToken, "accessToken");
+			ErrorUtilities.VerifyProtocol(accessToken.All(ch => accessTokenAuthorizationHeaderAllowedCharacters.IndexOf(ch) >= 0), OAuthStrings.AccessTokenInvalidForHttpAuthorizationHeader);
+
+			requestHeaders[HttpRequestHeader.Authorization] = string.Format(
 				CultureInfo.InvariantCulture,
 				Protocol.BearerHttpAuthorizationHeaderFormat,
 				accessToken);
