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

IPA-KDB: use relative path in ipa-certmap config snippet #672

Closed

Conversation

sumit-bose
Copy link
Contributor

@sumit-bose sumit-bose commented Mar 29, 2017

Architecture specific paths should be avoided in the global Kerberos
configuration because it is read e.g. by 32bit and 64bit libraries they
are installed in parallel.

Resolves https://pagure.io/freeipa/issue/6833

Architecture specific paths should be avoided in the global Kerberos
configuration because it is read e.g. by 32bit and 64bit libraries they
are installed in parallel.

Resolves https://pagure.io/freeipa/issue/6833
@tiran
Copy link
Member

tiran commented Mar 30, 2017

LGTM

For the recording: according to https://web.mit.edu/kerberos/krb5-1.12/doc/admin/conf_files/krb5_conf.html#plugins the plugin directive uses plugin_base_dir as base dir:

module
This tag may have multiple values. Each value is a string of the form modulename:pathname, which causes the shared object located at pathname to be registered as a dynamic module named modulename for the pluggable interface. If pathname is not an absolute path, it will be treated as relative to the plugin_base_dir value from [libdefaults].

plugin_base_dir
If set, determines the base directory where krb5 plugins are located. The default value is the krb5/plugins subdirectory of the krb5 library directory.

@sumit-bose What happens when the shared library is missing? Does 32bit kinit fail or work on a X86_64 system when 32bit ipadb.so is missing?

@abbra
Copy link
Contributor

abbra commented Mar 30, 2017

@sumit-bose What happens when the shared library is missing? Does 32bit kinit fail or work on a X86_64 system when 32bit ipadb.so is missing?

It is not about kinit. The module is for KDC, not client side. We guarantee it exists because we install it.

@ghost ghost self-assigned this Apr 4, 2017
@HonzaCholasta HonzaCholasta added the ack Pull Request approved, can be merged label Apr 5, 2017
@HonzaCholasta
Copy link
Contributor

master:

  • 6c2772d IPA-KDB: use relative path in ipa-certmap config snippet

ipa-4-5:

  • fa46a01 IPA-KDB: use relative path in ipa-certmap config snippet

@HonzaCholasta HonzaCholasta added the pushed Pull Request has already been pushed label Apr 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
4 participants