-
Notifications
You must be signed in to change notification settings - Fork 42
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
Configure zosmf-auth plugin during installation #398
Comments
This plugin requires configuration that is unnecessary these days, because apiml supports zosmf anyway. This does not do anything that apiml auth does not also do. Should we instead just remove this as a default plugin? We can still ship it, just not registered into the server. It is much easier to install plugins before, now that we have a script, so people can add it back easily. The reason why we pre-installed it was to avoid that step that used to be hard. |
We should not require unnecessary steps. Let's remove as a default plugin if no known application requires it. |
Sure. Here's the difference: Today (1.9):
This PR: zowe/zowe-install-packaging#1190
So essentially this trades off 1 extra step for no more warning. AFAIK nobody needs this step currently. This plugin only makes zosmf the authentication provider, but apiml already uses zosmf as the authentication provider, so its redundant. |
@jthyssenrocket commented on Mon Feb 10 2020
The zosmf-auth plugin is enabled by default but requires configuration by updating z/OSMF host and port in remote.json
Since the installer already provides the z/OSMF host and port during installation I suggest automatically updating z/OSMF auth remote.json.
If not configured warning message will be shown in the Zowe STC output which may confuse users:
@jplinardon commented on Mon Feb 10 2020
@Joe-Winchester, @stevenhorsman I don't see you as watcher on this repo, so FYI for planning/scheduling.
@1000TurquoisePogs FYI if the WebUI squad needs to do the work.
The text was updated successfully, but these errors were encountered: