[Fedora-directory-devel] Re: [Bug 244749] Configure Pass Thru Auth
by Noriko Hosoi
Summary: Configure Pass Thru Auth
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244749
The first diff is for the Admin Server setup and is almost identical to the proposal in comment #20, which is reviewed by Rich (Thank you!)
The second one is for the Directory Server ds_newinst; proposing to remove the dependency on the Configuration Directory from the DS code and moving them to the Admin Server code.
Thanks,
--noriko
------- Additional Comments From nhosoi(a)redhat.com 2007-06-27 17:57 EST -------
Created an attachment (id=158070)
--> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=158070&action=view)
cvs diffs (adminserver)
Files:
Makefile.am
admserv/newinst/src/AdminUtil.pm.in
admserv/newinst/src/dirserver.map.in
admserv/newinst/src/register_param.map.in
admserv/newinst/src/setup-ds-admin.pl.in
admserv/schema/ldif/15dspta.ldif.tmpl.in
Description: resurrected the code adding cn=Pass Through Authentication for
o=netscape. In addition to the one in comment #20, adding the calling code to
admserv/newinst/src/setup-ds-admin.pl.in in case the new server is
non-configuration DS.
------- Additional Comments From nhosoi(a)redhat.com 2007-06-27 18:01 EST -------
Created an attachment (id=158071)
--> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=158071&action=view)
cvs diff (ldapserver)
Files:
ldap/admin/src/create_instance.[ch]
Description:
1) removing the dependency on the config_ds
2) ds_newinst always adds "cn=Pass Through Authentication" with the
nsslapd-pluginEnabled value off.
15 years, 11 months
[Fedora-directory-devel] Please Review: (246020) Viewlog CGI needs to be modified to work with Console
by Nathan Kinder
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=246020
Resolves: bug 246020
Bug Description: The current viewlog CGI needs to be modified to be able
to work
properly with the Console.
The CGI currently only supports the GET request method. This works
for Admin
Express, but Console expects it to support the POST request method.
The output of the CGI currently includes a log selection combo box as
well as
other HTML elements that make for a nice useful log viewer page in Admin
Express. The Console needs to be able to get get the raw log content
that it
requests.
Reviewed by: ???
Files: see diff
Branch: HEAD
Fix Description: The attached diffs address the above issues. With
these changes,
the CGI will work with both the GET and POST request methods. I've
added a new
CGI parameter called "nohtml". This optional parameter is a flag that
allows the
caller to ask for only the raw content to be sent back. This
satisfies the needs
of the Console while keeping the behavior the same for Admin Express.
Platforms tested: FC6
Flag Day: no
Doc impact: no
https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=158068&action=diff
15 years, 11 months
[Fedora-directory-devel] Please Review: (245665) dsalib still relies on server root
by Nathan Kinder
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245665
Resolves: bug 245665
Bug Description: The dsa library in Admin Server still heavily relies on the
concept of a server root. This library is used by the Directory Server cgi
tasks.
The dsa library should try to get paths dynamically from the Directory
Server
config when possible. Other paths that are not in the config, such as the
location of command-line utilities, should be passed in as a define at
compilation time.
Reviewed by: ???
Files: see diff
Branch: HEAD
Fix Description: This pass eliminated the usage of a "server root" in
the dsa library.
I also refactored some of the code that was referring to the "install
root" as
it did not seem to really represent the directory that it refers to.
I renamed
it to "instance dir" and adjusted the getter function and variables
accordingly. I needed to pass in the DS libdir as a define for being
able to
find the instance dir in the code.
I also found that I was incorrectly using the lockdir config attribute
to set
the rundir, which was causing problems for the start, stop, and
restart CGIs.
We don't have the rundir in cn=config by default, but we already have
it set as
a PIDDIR define, so I opted to use that instead. The start, stop, and
restart
CGIs now work properly.
Platforms tested: FC6
Flag Day: no
Doc impact: no
https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=157923&action=diff
15 years, 11 months
[Fedora-directory-devel] Please review: Bug 245369: mod_admserv: Task cache refresh uses wrong credentials
by Rich Megginson
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245369
Resolves: bug 245369
Bug Description: mod_admserv: Task cache refresh uses wrong credentials
Reviewed by: ???
Files: see diff
Branch: HEAD
Fix Description: When the user requests a Task url, the admin server first
figures out which server instance (or product) the request is for, then
checks
to see if it has seen that server or product before. If not, it uses the
function sync_task_sie_data() to read the task data from the SIEs and ISIEs.
However, it needs to use the credentials of the currently authenticated user
to do so, because the tasks are protected by ACIs, and the user should
only be
allowed to read those tasks the user has access to. The interface to read
these tasks is not great. It expects the SIE is a user with a password, and
it attempts to bind as that user, instead of the currently authenticated
user.
I had to hack it to force it to use the current userdn and password instead
of the SIE DN and SIE password.
The SIE DN and password are now deprecated for binding. There were a couple
of places where the SIE was used for both the bind DN and the SIE DN. I've
created another structure member for the admservSieDN for use as the SIE
(the
configuration base DN) instead of as a bind DN, and deprecated the use
of the
SIE as the bind DN elsewhere in the code.
Platforms tested: RHEL4
Flag Day: no
Doc impact: no
https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=157632&action=diff
15 years, 11 months
[Fedora-directory-devel] Please review: Bug 245214: adminserver: Solaris port for 1.1
by Rich Megginson
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245214
Resolves: bug 245214
Bug Description: adminserver: Solaris port for 1.1
Reviewed by: ???
Files: see diff
Branch: HEAD
Fix Description: Fortitude can supply different components, depending on
the operating system. On Fedora, mod_nss and Apache are already there,
so we don't use Fortitude.
On RHEL4, we use Fortitude for mod_nss.
On Solaris, we use Fortitude for Apache and mod_nss.
I've created a fortitude.m4 that can handle all of these cases, and
changed the existing httpd.m4 and mod_nss.m4 to first see if
fortitude.m4 has found the component. configure will also pass the
proper --with flags to mod_admserv and mod_restartd.
ICU introduces a dependency on C++. Solaris requires linking with the
C++ libraries and that also requires using C++ to link as well. This
means using the CXXLINK macro.
There were a few other minor porting issues which required the
additional compiler defines and use of config.h.
We need to set the libpath before using the @cgibindir@/config program.
LIBPATH was not being set correctly on Solaris because I was using the
extended sort flags instead of the standard ones.
Platforms tested: Solaris 9 64bit
Flag Day: no
Doc impact: no
https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=157561&action=diff
15 years, 11 months