Skip to content

Support selecting a user from dropdown - #23

Open
apobbati wants to merge 2 commits into
mcguinness:masterfrom
codeblockio:master
Open

Support selecting a user from dropdown#23
apobbati wants to merge 2 commits into
mcguinness:masterfrom
codeblockio:master

Conversation

@apobbati

Copy link
Copy Markdown

Currently, there seems to be no easy way to select from a preset of users without actually entering the associated attributes. I'm creating an enhancement to make this work when an array of users is specified in the configuration file (config.js)

@apobbati

Copy link
Copy Markdown
Author

Adding an image for clarity:

image

@john-aws

john-aws commented Mar 27, 2018

Copy link
Copy Markdown
Contributor

While I think this PR is potentially useful, it seems to have (at least) a couple of problems:

  1. it doesn't sync the attribute statements when you select a user from the dropdown
  2. if you pick the 'Saml 2' user from the dropdown and click Sign In, the posted assertion that an SP sees is in error and emits "Error: SessionIndex not an attribute of AuthnStatement." because the added users have no SessionIndex

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants