Comparison

mod_http_oauth2: Add settings for allowed grant and response types So that you can opt-in to the insecure methods...

comparison

equal deleted inserted replaced

-:fa3059e653fa
+:6a3c1febd7be
 	response_type_handlers.token = nil;
 	grant_type_handlers.authorization_code = nil;
 	check_credentials = function () return false end
 end
+local allowed_grant_type_handlers = module:get_option_set("allowed_oauth2_grant_types", {"authorization_code", "password"})
+for handler_type in pairs(grant_type_handlers) do
+	if not allowed_grant_type_handlers:contains(handler_type) then
+		grant_type_handlers[handler_type] = nil;
+	end
+end
+-- "token" aka implicit flow is considered insecure
+local allowed_response_type_handlers = module:get_option_set("allowed_oauth2_response_types", {"code"})
+for handler_type in pairs(allowed_response_type_handlers) do
+	if not allowed_grant_type_handlers:contains(handler_type) then
+		grant_type_handlers[handler_type] = nil;
+	end
+end
 function handle_token_grant(event)
 	event.response.headers.content_type = "application/json";
 	local params = http.formdecode(event.request.body);
 	if not params then
 		return oauth_error("invalid_request");