Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Better support for enterprise logins #71

Open
regan-sarwas opened this issue Nov 3, 2016 · 2 comments
Open

Better support for enterprise logins #71

regan-sarwas opened this issue Nov 3, 2016 · 2 comments

Comments

@regan-sarwas
Copy link

We use a single custom template (https://www.nps.gov/gis/storymaps/mapjournal/v1/index.html) to create and view application items from many users (authorizedOwners: ["*"]). Most of the applications are public others, usually while under development, are not. This works great with basic (builtin) accounts, which are handled with the esri/arcgis/Portal.signIn() and the Identity Manager. Users are only asked to authenticate when viewing a private application, or when invoking builder (&edit). Unfortunately, Portal.signIn() does not appear to support enterprise accounts.

I understand Oauth is the preferred option (because it supports enterprise accounts?), however as stated in index.html, this will require all users to login, which is obviously unacceptable if we are serving an app to the general public.

Could portalLogin() be revised to use Oauth instead of Portal.signIn()?

@oevans
Copy link
Member

oevans commented Nov 3, 2016

Thanks for your feedback about this, @regan-sarwas. We will take a look at this for an upcoming release.

Until then you could deploy two copies of your custom template, one for viewing your public stories (without OAuth) and one for builder (with OAuth).

@regan-sarwas
Copy link
Author

After additional testing, I think the only thing that needs fixing is removing this comment in the index.html. In my testing, if the appid is shared with everyone, and a valid oAuthAppId and authorizedOwners: ["*"] are specified in the index.html, then viewers of the app are not required to authenticate.

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

No branches or pull requests

2 participants