Added parallelization of App Service requests

src/github.com/matrix-org/dendrite/appservice/routing/user.go

@@ -7,6 +7,8 @@ import (
 	"github.com/matrix-org/util"
 	"io/ioutil"
 	"net/http"
+	"net/url"
+	"sync"
 )
 
 // URIToUIDResponse represents response to an AppService URI to User Id
@@ -15,6 +17,8 @@ type URIToUIDResponse struct {
 	UserID string `json:"user_id"`
 }
 
+const PathPrefixUnstableThirdPartyUser = "/_matrix/app/unstable/thirdparty/user/"
+
 // URIToUID implements `/_matrix/app/r0/user?uri={url_encoded_uri}`, which
 // enables users to contact App Service users directly by taking an encoded
 // URI and turning it into a Matrix ID on the homeserver.
@@ -27,36 +31,51 @@ func URIToUID(req *http.Request, cfg config.Dendrite) util.JSONResponse {
 			JSON: nil,
 		}
 	}
+	// contact the app services in parallel, rather than sequentially
+	var wg sync.WaitGroup
+	userIDs := make([]string, 0)
 	for _, appservice := range cfg.Derived.ApplicationServices {
 		// Check all the fields associated with each application service
 		if !appservice.IsInterestedInUserID(uri) {
 			continue
 		}
-		// call the application service
+		wg.Add(1)
-		reqURL := "http://" + appservice.URL + "/_matrix/app/unstable/thirdparty/user/" +
+		// call the applicable application services in parallel
-			appservice.ID + "?access_token=" + appservice.HSToken +
+		go func(as config.ApplicationService, ids *[]string) {
-			"&fields=" + uri
+			defer wg.Done()
-		resp, err := http.Get(reqURL)
+			reqURL, err := url.Parse(as.URL + PathPrefixUnstableThirdPartyUser + as.ID +
+				"?access_token=" + as.HSToken +
+				"&fields=" + uri)
+			if err != nil {
+				return
+			}
+			resp, err := http.Get(reqURL.String())
 			// take the first successful match and send that back to the user
 			if err != nil {
-			continue
+				return
 			}
 			// decode the JSON to get the field we want
 			body, _ := ioutil.ReadAll(resp.Body)
 			respMap := map[string]interface{}{}
 			if err := json.Unmarshal(body, &respMap); err != nil {
-			return util.JSONResponse{
+				return
-				Code: http.StatusBadRequest,
-				JSON: jsonerror.NotJSON("The request body could not be decoded into valid JSON. " + err.Error()),
-			}
 			}
 			if userID, ok := respMap["userid"].(string); ok {
+				*ids = append(*ids, userID)
+			}
+		}(appservice, &userIDs)
+
+	}
+
+	wg.Wait()
+
+	if len(userIDs) > 0 {
 		return util.JSONResponse{
 			Code: http.StatusOK,
-				JSON: URIToUIDResponse{UserID: userID},
+			JSON: URIToUIDResponse{UserID: userIDs[0]},
-			}
 		}
 	}
+
 	return util.JSONResponse{
 		Code: http.StatusNotFound,
 		JSON: jsonerror.NotFound("URI not supported by app services"),