lbehm/workadventure
1
0
Fork 0
Code Issues Pull requests Releases Activity

Merge pull request #1607 from thecodingmachine/fix_query_string

Browse source 
Fixing "Query string parameters prevent WA from loading"
This commit is contained in:
David Négrier 2021-11-30 14:48:15 +01:00 committed by GitHub
parent 21e400a4a1 06483fd586
commit 986f826830
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
front/src/Connexion/ConnectionManager.ts
View file

@ -163,12 +163,13 @@ class ConnectionManager {
console.error(err);
}
} else {
const query = urlParams.toString();
roomPath =
window.location.protocol +
"//" +
window.location.host +
window.location.pathname +
urlParams.toString() + //use urlParams because the token param must be deleted
(query ? "?" + query : "") + //use urlParams because the token param must be deleted
window.location.hash;
}

3
tests/tests/test.ts
View file

@ -4,8 +4,9 @@ const fs = require('fs')
import { Selector } from 'testcafe';
import {userAlice} from "./utils/roles";
// Note: we are also testing that we can connect if the URL contains a random query string
fixture `Variables`
.page `http://play.workadventure.localhost/_/global/maps.workadventure.localhost/tests/Variables/Cache/variables_tmp.json`;
.page `http://play.workadventure.localhost/_/global/maps.workadventure.localhost/tests/Variables/Cache/variables_tmp.json?somerandomparam=1`;
test("Test that variables cache in the back don't prevent setting a variable in case the map changes", async (t: TestController) => {
// Let's start by visiting a map that DOES not have the variable.