Skip to content

Commit 2002715

Browse files
Replace legacy url Node module with new URL API, bringing it in-line with the Quickstart
1 parent 316100c commit 2002715

File tree

1 file changed

+5
-10
lines changed

1 file changed

+5
-10
lines changed

index.js

+5-10
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ const hash = require("pbkdf2-password")();
1717
const path = require("path");
1818
const session = require("express-session");
1919
const jwt = require("jsonwebtoken");
20-
const url = require("url");
2120

2221
var app = (module.exports = express());
2322

@@ -189,15 +188,11 @@ app.post("/login", function (req, res, next) {
189188
});
190189

191190
app.get("/sso/metabase", restrict, (req, res) => {
192-
res.redirect(
193-
url.format({
194-
pathname: `${METABASE_SITE_URL}/auth/sso`,
195-
query: {
196-
jwt: signUserToken(req.session.user),
197-
return_to: `${req.query.return_to || "/"}?${mods}`,
198-
},
199-
})
200-
);
191+
const ssoUrl = new URL("/auth/sso", METABASE_SITE_URL);
192+
ssoUrl.searchParams.set("jwt", signUserToken(req.session.user));
193+
ssoUrl.searchParams.set("return_to", `${req.query.return_to ?? "/"}?${mods}`);
194+
195+
res.redirect(ssoUrl);
201196
});
202197

203198
const PORT = 8080;

0 commit comments

Comments
 (0)