Bug #2558
closed
(ex PT/ Technique) User Management: The fullname of the user is only set at creation. It will not be updated or checked.
Added by Nicolas PERRON almost 12 years ago.
Updated over 11 years ago.
Description
If I create a user without fullname and change the Technique to update its name to 'Brandon', it will return success even if userQA1 has no fullname. It will be the same if the fullname is wrong.
Furthermore, checking a user with a wrong fullname will return success too.
it seems that fullname is never checked or updated.
- Status changed from New to In progress
Full name is not checked and only set at creation with an argument to useradd.
- Target version changed from 2.3.8 to 2.3.9
- Status changed from In progress to Pending technical review
- % Done changed from 0 to 100
Now, fullname is checked and changed if defined in the Technique
Well it looks correct, I approve.
BUT !
I have to little objections:
- You are editing /etc/passwd, which means your implementation is Linux and some BSDs-centric.
- Editing /etc/passwd is quite dangerous, we should test this thoroughly.
Matthieu CERDA wrote:
Well it looks correct, I approve.
BUT !
I have to little objections:
- You are editing /etc/passwd, which means your implementation is Linux and some BSDs-centric.
Actually, we are concentrated on Linux implementation since Windows implimentation is quite wonky.
- Editing /etc/passwd is quite dangerous, we should test this thoroughly.
How can we test /etc/passwd ? The previous use of this Technique didn't made any check on /etc/passwd, /etc/shadow or /etc/group
Nicolas PERRON wrote:
Matthieu CERDA wrote:
Well it looks correct, I approve.
BUT !
I have to little objections:
- You are editing /etc/passwd, which means your implementation is Linux and some BSDs-centric.
Actually, we are concentrated on Linux implementation since Windows implimentation is quite wonky.
- Editing /etc/passwd is quite dangerous, we should test this thoroughly.
How can we test /etc/passwd ? The previous use of this Technique didn't made any check on /etc/passwd, /etc/shadow or /etc/group
Please make a new issue about a new implementation.
I do not think a new issue is useful, I just wanted to point out that editing /etc/passwd is a thing that should be tested functionally, not in the PT.
About the Windows support, yep should keep it in mind, but it is obviously not currently on schedule.
- Status changed from Pending technical review to Released
This looks valid. However, I wonder if it might not have been a bit more cfenginesque to use the bundle edit_line set_colon_field of the stdlib, and define classes in the files: promises part rather than in the bundle edit_line
Also available in: Atom
PDF
Updated by Nicolas PERRON 
Updated by 
Updated by Matthieu CERDA 
Updated by