Login
[x]
Log in using an account from:
Fedora Account System
Red Hat Associate
Red Hat Customer
Or login using a Red Hat Bugzilla account
Forgot Password
Login:
Hide Forgot
Create an Account
Red Hat Bugzilla – Attachment 317002 Details for
Bug 459729
Windows sync support in IPA
[?]
New
Simple Search
Advanced Search
My Links
Browse
Requests
Reports
Current State
Search
Tabular reports
Graphical reports
Duplicates
Other Reports
User Changes
Plotly Reports
Bug Status
Bug Severity
Non-Defaults
|
Product Dashboard
Help
Page Help!
Bug Writing Guidelines
What's new
Browser Support Policy
5.0.4.rh83 Release notes
FAQ
Guides index
User guide
Web Services
Contact
Legal
This site requires JavaScript to be enabled to function correctly, please enable it.
[patch]
fix issues brought up by code review
0011-fix-issues-brought-up-by-initial-review-of-ipa-winsy.patch (text/plain), 3.76 KB, created by
Rich Megginson
on 2008-09-17 19:15:38 UTC
(
hide
)
Description:
fix issues brought up by code review
Filename:
MIME Type:
Creator:
Rich Megginson
Created:
2008-09-17 19:15:38 UTC
Size:
3.76 KB
patch
obsolete
>From e5156ca48e287f6a6c30ba6481da9914220e183c Mon Sep 17 00:00:00 2001 >From: Rich Megginson <rmeggins@redhat.com> >Date: Wed, 17 Sep 2008 13:13:42 -0600 >Subject: [PATCH] fix issues brought up by initial review of ipa winsync enhancements > >--- > ipa-server/ipa-install/ipa-replica-manage | 20 +++++++++++++------- > ipa-server/ipa-install/share/indices.ldif | 13 +++++++++++++ > .../ipa-winsync/ipa-winsync-config.c | 4 +++- > 3 files changed, 29 insertions(+), 8 deletions(-) > >diff --git a/ipa-server/ipa-install/ipa-replica-manage b/ipa-server/ipa-install/ipa-replica-manage >index 2021eab..c8e5735 100644 >--- a/ipa-server/ipa-install/ipa-replica-manage >+++ b/ipa-server/ipa-install/ipa-replica-manage >@@ -93,19 +93,25 @@ def del_master(replman, hostname): > > def add_master(replman, hostname, options): > other_args = {} >- if options.winsync: >- # these are the parameters required to create a winsync agreement >- other_args['winsync'] = True >- if options.port: >- other_args['port'] = options.port >+ if options.port: >+ other_args['port'] = options.port >+ if options.binddn: > other_args['binddn'] = options.binddn >+ if options.bindpw: > other_args['bindpw'] = options.bindpw >+ if options.cacert: > other_args['cacert'] = options.cacert >- # have to install the windows ca cert before doing anything else >+ if options.winsync: >+ other_args['winsync'] = True >+ if not options.binddn or not options.bindpw or not options.cacert: >+ logging.error("The arguments --binddn, --bindpw, and --cacert are required to create a winsync agreement") >+ sys.exit(1) >+ if options.cacert: >+ # have to install the given CA cert before doing anything else > ds = dsinstance.DsInstance(realm_name = get_realm_name(), > dm_password = replman.dirman_passwd) > if not ds.add_ca_cert(options.cacert): >- logging.error("Could not load the required CA certificate file [%s] - cannot add winsync agreement" % >+ logging.error("Could not load the required CA certificate file [%s]" % > options.cacert) > sys.exit(1) > # have to reconnect replman connection since the directory server was restarted >diff --git a/ipa-server/ipa-install/share/indices.ldif b/ipa-server/ipa-install/share/indices.ldif >index 632a287..e008983 100644 >--- a/ipa-server/ipa-install/share/indices.ldif >+++ b/ipa-server/ipa-install/share/indices.ldif >@@ -82,3 +82,16 @@ nsSystemIndex:false > nsIndexType:eq > nsMatchingRule: integerOrderingMatch > >+dn: cn=ntUniqueId,cn=index,cn=userRoot,cn=ldbm database,cn=plugins,cn=config >+objectClass: top >+objectClass: nsIndex >+cn: ntUniqueId >+nsSystemIndex: false >+nsIndexType: eq,pres >+ >+dn: cn=ntUserDomainId,cn=index,cn=userRoot,cn=ldbm database,cn=plugins,cn=config >+objectClass: top >+objectClass: nsIndex >+cn: ntUserDomainId >+nsSystemIndex: false >+nsIndexType: eq,pres >diff --git a/ipa-server/ipa-slapi-plugins/ipa-winsync/ipa-winsync-config.c b/ipa-server/ipa-slapi-plugins/ipa-winsync/ipa-winsync-config.c >index 7020847..f4c4484 100644 >--- a/ipa-server/ipa-slapi-plugins/ipa-winsync/ipa-winsync-config.c >+++ b/ipa-server/ipa-slapi-plugins/ipa-winsync/ipa-winsync-config.c >@@ -110,7 +110,9 @@ ipa_winsync_config(Slapi_Entry *config_e) > > if ( inited ) { > slapi_log_error( SLAPI_LOG_FATAL, IPA_WINSYNC_PLUGIN_NAME, >- "only one PAM pass through plugin instance can be used\n" ); >+ "Error: IPA WinSync plug-in already configured. " >+ "Please remove the plugin config entry [%s]\n", >+ slapi_entry_get_dn_const(config_e)); > return( LDAP_PARAM_ERROR ); > } > >-- >1.5.5.1 >
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
Attachments on
bug 459729
:
314729
|
314730
|
314731
|
314732
|
314733
|
314844
|
314845
|
316460
| 317002 |
317807
|
318214
|
319412
|
319413
|
319414
|
319415
|
319416
|
319786
|
320039
|
320040