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

Incorrect values provided for service account environment in unattended config template #8

Closed
mikkonie opened this issue Jan 27, 2022 · 3 comments
Labels
bug Something isn't working

Comments

@mikkonie
Copy link
Contributor

I noticed that we set PAM auth and SSL for the service account environment if they are used for incoming connections.

Unless I'm mistaken, this is wrong: the (mostly) locally connecting default service account uses native login and SSL should not be necessary, right?

I'll verify this with our existing setup, but if I'm right I'll need to fix this asap.

@mikkonie mikkonie added the bug Something isn't working label Jan 27, 2022
@messersc
Copy link

I was wondering, too.

As far as I understand, you are correct and the native auth does not need SSL.

But why would it be harmful to have pam and SSL set-up already? Ideally, you would have the service user locally and remote users through PAM with LDAP.

mikkonie added a commit that referenced this issue Jan 27, 2022
@mikkonie
Copy link
Contributor Author

But why would it be harmful to have pam and SSL set-up already? Ideally, you would have the service user locally and remote users through PAM with LDAP.

That is exactly how it is done. This ticket is specifically about the service account environment, where the login should be native, not pam. AFAIK the SSL settings can still be there though.

I fixed this issue, won't close though until I've properly tested it together with #9.

@mikkonie
Copy link
Contributor Author

Seems I have forgotten to close this issue earlier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants