annotate mod_s2s_reload_newcomponent/mod_s2s_reload_newcomponent.lua @ 5418:f2c7bb3af600

mod_http_oauth2: Add role selector to consent page List includes all roles available to the user, if more than one. Defaults to either the first role in the scope string or the users primary role. Earlier draft listed all roles, but having options that can't be selected is bad UX and the entire list of all roles on the server could be long, and perhaps even sensitive. Allows e.g. picking a role with fewer permissions than what might otherwise have been selected. UX wise, doing this with more checkboxes or possibly radio buttons would have been confusion and/or looked messier. Fixes the previous situation where unselecting a role would default to the primary role, which could be more permissions than requested.
author Kim Alvefur <zash@zash.se>
date Fri, 05 May 2023 01:23:13 +0200
parents fe5bb7b13a59
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
213
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
1 local modulemanager = require "core.modulemanager";
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
2 local config = require "core.configmanager";
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
3
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
4 module.host = "*";
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
5
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
6 local function reload_components()
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
7 local defined_hosts = config.getconfig();
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
8
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
9 for host in pairs(defined_hosts) do
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
10 if (not hosts[host] and host ~= "*") then
214
7487f8b47662 mod_s2s_reload_newcomponent: fix debug logs
Gaurav <gauravsri@gmail.com>
parents: 213
diff changeset
11 module:log ("debug", "loading new component %s", host);
2781
fe5bb7b13a59 mod_s2s_reload_newcomponent: Fix to use imported configmanager instead of global
Matthew Wild <mwild1@gmail.com>
parents: 214
diff changeset
12 modulemanager.load(host, config.get(host, "core", "component_module"));
213
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
13 end
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
14 end;
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
15
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
16 return;
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
17 end
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
18
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
19 module:hook("config-reloaded", reload_components);
89051a926f74 initial creation of module for reloading new components
Gaurav <gauravsri@gmail.com>
parents:
diff changeset
20