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

Bugfixes and features for v1.7.1 #114

Merged
merged 1 commit into from Nov 26, 2019
Merged

Conversation

@jordanfilteau1995
Copy link
Contributor

jordanfilteau1995 commented Nov 21, 2019

Signed-off-by: Jordan Filteau jfilteau@rocketsoftware.com

Signed-off-by: Jordan Filteau <jfilteau@rocketsoftware.com>
@jordanfilteau1995

This comment has been minimized.

Copy link
Contributor Author

jordanfilteau1995 commented Nov 25, 2019

This fixes the following bugs:

  • It would cut off the end of an acid when getting the access list of a group, resulting in invalid output in the response.

  • All of the different administrator suffixes weren't defined, so it was incorrectly returning administrators.

This adds the following features:

  • A backup routine for when a non-administrator tries to access the API. Instead of executing privileged commands and failing, it will execute a command to get their profile, and return only the information in their scope. This is a feature that most people won't need, since you'd ideally want to be an administrator if you were using this API, but the functionality is there.

  • The ability to retrieve profiles only by prefix. This can be done by looking for a profile with a "." at the end. This will act as a wildcard which extracts everything matching that prefix.

@jordanfilteau1995

This comment has been minimized.

Copy link
Contributor Author

jordanfilteau1995 commented Nov 25, 2019

These fixes are important in maintaining consistency with the API when called from a system with RACF.

@1000TurquoisePogs

This comment has been minimized.

Copy link
Member

1000TurquoisePogs commented Nov 26, 2019

I believe this is a repeat of #112 but for RC, which is fine.

@1000TurquoisePogs 1000TurquoisePogs merged commit 349968a into zowe:rc Nov 26, 2019
2 checks passed
2 checks passed
DCO DCO
Details
continuous-integration/jenkins/pr-merge This commit looks good
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.