Skip to content
Snippets Groups Projects
Commit a867ecbc authored by Lloyd Hilaiel's avatar Lloyd Hilaiel
Browse files

merge issuer hotfix into production

parents 95e9fc58 0345464c
No related branches found
No related tags found
No related merge requests found
......@@ -12,6 +12,9 @@ const https = require("https");
const url = require("url");
const rsa = require("./rsa.js");
// configuration information to check the issuer
const config = require("../../libs/configuration.js");
var Webfinger = (function() {
if (!console) console = {};
......@@ -269,6 +272,14 @@ IDAssertion.prototype =
return;
}
// check that the issuer is just US for now, no other issuer
// FIXME: this will need to change for certs
var expected_issuer = config.get('hostname') + ':' + config.get('port');
if (payload.issuer != expected_issuer) {
onError("Issuer can only be ourselves for now, it should be: " + expected_issuer);
return;
}
// (if there was a certificate, we could verify it here)
// but for now we will assume email-based lookup
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment