Skip to content

Commit b9b03fc

Browse files
committed
Renaming HTTP 'key' query parameter to 'code' (consistency with WebHooks)
1 parent 4ba5225 commit b9b03fc

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

src/WebJobs.Script.WebHost/Filters/AuthorizationLevelAttribute.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ internal static AuthorizationLevel GetAuthorizationLevel(HttpRequestMessage requ
6464
else
6565
{
6666
var queryParameters = request.GetQueryNameValuePairs().ToDictionary(p => p.Key, p => p.Value, StringComparer.OrdinalIgnoreCase);
67-
queryParameters.TryGetValue("key", out keyValue);
67+
queryParameters.TryGetValue("code", out keyValue);
6868
}
6969

7070
if (!string.IsNullOrEmpty(keyValue))

test/WebJobs.Script.Tests/AuthorizationLevelAttributeTests.cs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -151,9 +151,9 @@ public void GetAuthorizationLevel_InvalidKeyHeader_ReturnsAnonymous()
151151
}
152152

153153
[Fact]
154-
public void GetAuthorizationLevel_ValidKeyQueryParam_MasterKey_ReturnsAdmin()
154+
public void GetAuthorizationLevel_ValidCodeQueryParam_MasterKey_ReturnsAdmin()
155155
{
156-
Uri uri = new Uri(string.Format("http://functions/api/foo?key={0}", testMasterKeyValue));
156+
Uri uri = new Uri(string.Format("http://functions/api/foo?code={0}", testMasterKeyValue));
157157
HttpRequestMessage request = new HttpRequestMessage(HttpMethod.Get, uri);
158158

159159
AuthorizationLevel level = AuthorizationLevelAttribute.GetAuthorizationLevel(request, _mockSecretManager.Object);
@@ -162,24 +162,24 @@ public void GetAuthorizationLevel_ValidKeyQueryParam_MasterKey_ReturnsAdmin()
162162
}
163163

164164
[Fact]
165-
public void GetAuthorizationLevel_ValidKeyQueryParam_FunctionKey_ReturnsFunction()
165+
public void GetAuthorizationLevel_ValidCodeQueryParam_FunctionKey_ReturnsFunction()
166166
{
167167
// first try host level function key
168-
Uri uri = new Uri(string.Format("http://functions/api/foo?key={0}", testHostFunctionKeyValue));
168+
Uri uri = new Uri(string.Format("http://functions/api/foo?code={0}", testHostFunctionKeyValue));
169169
HttpRequestMessage request = new HttpRequestMessage(HttpMethod.Get, uri);
170170
AuthorizationLevel level = AuthorizationLevelAttribute.GetAuthorizationLevel(request, _mockSecretManager.Object, functionName: "TestFunction");
171171
Assert.Equal(AuthorizationLevel.Function, level);
172172

173-
uri = new Uri(string.Format("http://functions/api/foo?key={0}", testFunctionKeyValue));
173+
uri = new Uri(string.Format("http://functions/api/foo?code={0}", testFunctionKeyValue));
174174
request = new HttpRequestMessage(HttpMethod.Get, uri);
175175
level = AuthorizationLevelAttribute.GetAuthorizationLevel(request, _mockSecretManager.Object, functionName: "TestFunction");
176176
Assert.Equal(AuthorizationLevel.Function, level);
177177
}
178178

179179
[Fact]
180-
public void GetAuthorizationLevel_InvalidKeyQueryParam_ReturnsAnonymous()
180+
public void GetAuthorizationLevel_InvalidCodeQueryParam_ReturnsAnonymous()
181181
{
182-
Uri uri = new Uri(string.Format("http://functions/api/foo?key={0}", "invalid"));
182+
Uri uri = new Uri(string.Format("http://functions/api/foo?code={0}", "invalid"));
183183
HttpRequestMessage request = new HttpRequestMessage(HttpMethod.Get, uri);
184184

185185
AuthorizationLevel level = AuthorizationLevelAttribute.GetAuthorizationLevel(request, _mockSecretManager.Object);

0 commit comments

Comments
 (0)