Allow slashes unquoted in room names. This brings nested rooms.

src/app/spreed-webrtc-server/main.go

@@ -352,6 +352,8 @@ func runner(runtime phoenix.Runtime) error {
 	r.Handle("/robots.txt", http.StripPrefix(config.B, http.FileServer(http.Dir(path.Join(rootFolder, "static")))))
 	r.Handle("/favicon.ico", http.StripPrefix(config.B, http.FileServer(http.Dir(path.Join(rootFolder, "static", "img")))))
 	r.Handle("/ws", makeWSHandler(statsManager, sessionManager, codec, channellingAPI))
+
+	// Simple room handler.
 	r.HandleFunc("/{room}", httputils.MakeGzipHandler(roomHandler))
 
 	// Add API end points.
@@ -382,6 +384,10 @@ func runner(runtime phoenix.Runtime) error {
 		}
 	}
 
+	// Map everything else to a room when it is a GET.
+	rooms := r.PathPrefix("/").Methods("GET").Subrouter()
+	rooms.HandleFunc("/{room:.*}", httputils.MakeGzipHandler(roomHandler))
+
 	return runtime.Start()
 }
 

static/js/app.js

@@ -154,7 +154,7 @@ define([
 			$compileProvider.aHrefSanitizationWhitelist(/^\s*(https?|ftp|mailto|tel|file|filesystem|blob):/);
 			$compileProvider.imgSrcSanitizationWhitelist(/^\s*(https?|ftp|file|filesystem|blob):|data:image\//);
 			// Setup routing
-			$routeProvider.when("/:room", {});
+			$routeProvider.when("/:room*", {});
 			// Use HTML5 routing.
 			$locationProvider.html5Mode(true);
 		}]);

static/js/services/resturl.js

@@ -20,21 +20,42 @@
  */
 
 "use strict";
-define([
+define(["underscore"], function(_) {
-], function() {
 
+	// restURL
 	return ["globalContext", "$window", function(context, $window) {
-		return {
+		var RestURL = function() {};
-			room: function(id) {
+		RestURL.prototype.room = function(name) {
-				id = $window.encodeURIComponent(id);
+			var url = this.encodeRoomURL(name);
-				return $window.location.protocol + '//' + $window.location.host + context.Cfg.B + id;
+			return $window.location.protocol + '//' + $window.location.host + context.Cfg.B + url;
-			},
+		};
-			buddy: function(id) {
+		RestURL.prototype.buddy = function(id) {
-				return $window.location.protocol + '//' + $window.location.host + context.Cfg.B + "static/img/buddy/s46/" + id;
+			return $window.location.protocol + '//' + $window.location.host + context.Cfg.B + "static/img/buddy/s46/" + id;
-			},
+		};
-			api: function(path) {
+		RestURL.prototype.api = function(path) {
-				return (context.Cfg.B || "/") + "api/v1/" + path;
+			return (context.Cfg.B || "/") + "api/v1/" + path;
+		};
+		RestURL.prototype.encodeRoomURL = function(name, prefix) {
+			// Split parts so slashes are allowed.
+			var parts = name.split("/");
+			var url = [];
+			if (typeof prefix !== "undefined") {
+				url.push(prefix);
+			}
+			// Allow some thing in room name parts.
+			_.each(parts, function(p) {
+				p = $window.encodeURIComponent(p);
+				p = p.replace(/^%40/, "@");
+				p = p.replace(/^%24/, "$");
+				p = p.replace(/^%2B/, "+");
+				url.push(p);
+			});
+			if (url.length > 1 && url[url.length-1] === "") {
+				// Remove trailing slash if any.
+				url.pop();
 			}
+			return url.join("/");
 		};
+		return new RestURL();
 	}];
 });

static/js/services/rooms.js

@@ -103,7 +103,7 @@ define([
 			currentRoom = room;
 			if (priorRoom) {
 				priorRoomName = priorRoom.Name;
-				console.log("Left room", priorRoom.Name);
+				console.log("Left room", [priorRoom.Name]);
 				$rootScope.$broadcast("room.left", priorRoom.Name);
 			}
 			if (currentRoom) {
@@ -212,13 +212,10 @@ define([
 				return canJoinRooms;
 			},
 			joinByName: function(name, replace) {
-				name = $window.encodeURIComponent(name);
+				var url = restURL.encodeRoomURL(name, "");
-				name = name.replace(/^%40/, "@");
+				// Apply new URL.
-				name = name.replace(/^%24/, "$");
-				name = name.replace(/^%2B/, "+");
-
 				safeApply($rootScope, function(scope) {
-					$location.path("/" + name);
+					$location.path(url);
 					if (replace) {
 						$location.replace();
 					}