home folder and ssl sert #21
Conversation
Added selenium home folder managment and no SSL cert check.
| @@ -24,7 +24,8 @@ | |||
| include wget | |||
|
|
|||
| user { $user: | |||
| gid => $group, | |||
| gid => $group, | |||
| managehome => true, | |||
There was a problem hiding this comment.
Why is this needed? I'd rather that the role user optionally be defiend external to the module rather than adding a bunch user management features.
|
Just FYI - it's considered bad form to have to unrelated changes as part of the same commit. |
| timeout => $download_timeout, | ||
| execuser => $user, | ||
| require => File[$jar_path], | ||
| nocheckcertificate => true, |
There was a problem hiding this comment.
What problem is this trying to resolve? https://selenium-release.storage.googleapis.com/ is returning a public CA signed cert.
|
@EvgenIn ping? |
|
I apologize for the delayed answer. Didn't notice the previous msg. |
|
That would be great. |
|
It seems that all is working now, without any errors. I now think that the certificate issue was on the server side, not the client. I guess this change can be ignored now. |
|
That or a MITM attempt (seems unlikely on AWS) would be my guess. What about the change to the |
|
To my knowledge, selenium user must have a home folder for the server to function. PS: Creating the 'selenium' user as a system user sounds like a good idea. |
|
@jhoblitt i agree with @EvgenIn maybe create system user 'selenium' was best decision because into defaults params use user/group 'selenium' but this user/group not exist into system. i using the vagrant&puppet (vagrant use virtualbox centos6.5-64) and i set @EvgenIn if your use Amazon EC2 should set default ec2 user/group 'ec2-user' it work well for me. anyway create system user 'selenium' or not it decision for @jhoblitt to be, or not to be, creating the system user 'selenium' into this module |
|
@EvgenIn / @zivan user/group management was made optional by 2911fa6 allowing management of the user and/or group resource(s) external to the module. This is is vastly more flexible than trying to expose parameters to those resources via the selenium class. I'm going to go ahead and close this PR out (yikes has it been open for a long time). I would be happy to merge a PR to set the system attribute on the default user/group resources. |
Added selenium home folder managment and no SSL cert check.
puppet-lint alingment