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.

Sign up for GitHub

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

Jump to bottom

feat(jans-core): remove UPDATE_USER and USER_REGISTRATION scripts #1289

Closed
yuriyz opened this issue May 5, 2022 · 1 comment
Closed

feat(jans-core): remove UPDATE_USER and USER_REGISTRATION scripts #1289

yuriyz opened this issue May 5, 2022 · 1 comment
Assignees
@yuriyz
Labels
comp-jans-auth-server Component affected by issue or PR comp-jans-core Component affected by issue or PR kind-enhancement Issue or PR is an enhancement to an existing functionality kind-feature Issue or PR is a new feature request

Comments

@yuriyz
Copy link
Contributor

yuriyz commented May 5, 2022 •
edited
Loading

Description

feat(jans-core): remove UPDATE_USER and USER_REGISTRATION scripts

(Mike's request)

we should remove UPDATE_USER and USER_REGISTRATION
its a legacy feature
The best way to do this is via SCIM now
We don’t want the admin UI to be internet facing
UPDATE_USER was used from the admin UI, but it’s not a good idea
At this point, the admin ui uses SCIM for user mgt. anyway
@yuriyz yuriyz added kind-enhancement Issue or PR is an enhancement to an existing functionality comp-jans-auth-server Component affected by issue or PR labels May 5, 2022
@yuriyz yuriyz self-assigned this May 5, 2022
@mo-auto mo-auto added comp-jans-core Component affected by issue or PR kind-feature Issue or PR is a new feature request labels May 5, 2022
yuriyz added a commit that referenced this issue May 5, 2022
@yuriyz
feat(jans-core): remove UPDATE_USER and USER_REGISTRATION scripts #1289
c34e75d
@yuriyz yuriyz mentioned this issue May 5, 2022
Closed
@yuriyzz yuriyzz mentioned this issue May 5, 2022
Merged
yuriyz added a commit that referenced this issue May 5, 2022
@yuriyz
Merge pull request #1291 from JanssenProject/jans-auth-server-1289
0140932 
feat(jans-core): removed UPDATE_USER and USER_REGISTRATION scripts #1289
@yuriyz
Copy link
Contributor Author

yuriyz commented May 5, 2022

Both are removed.

@yuriyz yuriyz closed this as completed May 5, 2022
This was referenced May 18, 2022
Merged
Merged
Merged
Merged
Merged
This was referenced May 18, 2022
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yuriyz yuriyz

Labels
comp-jans-auth-server Component affected by issue or PR comp-jans-core Component affected by issue or PR kind-enhancement Issue or PR is an enhancement to an existing functionality kind-feature Issue or PR is a new feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yuriyz @mo-auto