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

Strange attribute values in admin entry #394

Closed
jgomer2001 opened this issue Dec 22, 2017 · 6 comments
Closed

Strange attribute values in admin entry #394

jgomer2001 opened this issue Dec 22, 2017 · 6 comments
Assignees
@yurem
Labels
exclude from release notes Issue found during QA or otherwise non-important issue to report in release notes question
Milestone
5.0

Comments

@jgomer2001
Copy link
Contributor

Recently I installed RC3. My admin user has some attributes I haven't seen set in other Gluu versions

For instance it has picture, profile, telephoneNumber, website, zoneinfo, c, phoneNumberVerified, gender, emailVerified

I am attaching a ldif file... I wonder why I have "America/Los_Angeles" in zone, or country=US if upon installation I did provide a different value for my location. Also the phone number... and birthday... it's weird.

Can someone replicate?

admin.ldif.txt
@jgomer2001 jgomer2001 added exclude from release notes Issue found during QA or otherwise non-important issue to report in release notes question labels Dec 22, 2017
@jgomer2001 jgomer2001 added this to the 3.1.2 milestone Dec 22, 2017
@yurem
Copy link
Contributor

yurem commented Dec 24, 2017

We changed this template 4 month ago.

@yurem yurem closed this as completed Dec 24, 2017
@yurem
Copy link
Contributor

yurem commented Dec 24, 2017

Or testes depends on this user values. We can create another admin user and move this user ldif to test data. Any ideas about new admin user uid?

@yurem yurem reopened this Dec 24, 2017
@yurem yurem modified the milestones: 3.1.2, 3.2.0 Dec 24, 2017
@yurem
Copy link
Contributor

yurem commented Jan 2, 2018

@yuriyz @qbert2k I offer to change in our tests test user 'admin' to 'test' (or any other name). After that we can remove not required fields from admin user entry in CE

@willow9886 willow9886 modified the milestones: 4.0, 4.1 Apr 22, 2019
@yuriyz yuriyz modified the milestones: 4.1, 4.2 Dec 12, 2019
@yurem yurem modified the milestones: 4.2, 5.0 May 26, 2020
@devrimyatar
Copy link
Collaborator

@yurem Shall I copy current admin user as testadmin to test data and remove those attributes mentione by Jose?

@yurem
Copy link
Contributor

yurem commented Jan 3, 2022

@mbaser yes, we can do this in master branch

devrimyatar added a commit that referenced this issue Jan 4, 2022
@devrimyatar
feat: testadmin user (ref: #394)
a7aa632
yurem pushed a commit that referenced this issue Jan 4, 2022
@devrimyatar
feat: testadmin user (ref: #394) (#801)
a01eab4 
* feat: testadmin user (ref: #394)

* fix: code smell
@devrimyatar
Copy link
Collaborator

Done

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

@yurem yurem

Labels
exclude from release notes Issue found during QA or otherwise non-important issue to report in release notes question
Projects
None yet
Milestone
5.0
Development

No branches or pull requests
5 participants
@yuriyz @willow9886 @jgomer2001 @yurem @devrimyatar