Skip to content

Commit

Permalink
Don't treat Juniper 'realm' field as authgroup
Browse files Browse the repository at this point in the history
The only point in the authgroup thing for the UI is to allow OpenConnect
to re-request the set of query fields for the new group. But the Juniper
protocol doesn't do that, and it's causing problems.

Signed-off-by: David Woodhouse <dwmw2@infradead.org>
  • Loading branch information
dwmw2 committed Aug 9, 2018
1 parent f5b0922 commit b0a50a7
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions auth-juniper.c
Expand Up @@ -188,8 +188,6 @@ static int parse_select_node(struct openconnect_info *vpninfo, struct oc_auth_fo
xmlnode_get_prop(node, "name", &opt->form.name);
opt->form.label = strdup(opt->form.name);
opt->form.type = OC_FORM_OPT_SELECT;
if (!strcmp(opt->form.name, "realm"))
form->authgroup_opt = opt;

for (child = node->children; child; child = child->next) {
struct oc_choice **new_choices;
Expand Down

0 comments on commit b0a50a7

Please sign in to comment.