Project

General

Profile

Bug #7409

Cannot read an auth source ldap's host via the API

Added by Dominic Cleal almost 4 years ago. Updated about 1 month ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
API
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1140313
Description of problem:
Host information is not returned when issuing an HTTP GET request to api/v2/auth_source_ldaps/:id. This does not make sense:

  • The "host" parameter must be specified when creating an auth source ldap. If this parameter is necessary when creating an auth source ldap, shouldn't it be possible to read back that information later?
  • The API documentation of theforeman.org provides an example of issuing an HTTP GET to api/v2/auth_source_ldaps/:id. The "host" parameter is returned.

Version-Release number of selected component (if applicable):
I tested this issue against qe-sat6-rhel65, which is running Satellite 6.0.4. The following software is installed on that machine:

  • apr-util-ldap-1.3.9-3.el6_0.1.x86_64
  • candlepin-0.9.23-1.el6_5.noarch
  • candlepin-common-1.0.1-1.el6_5.noarch
  • candlepin-scl-1-5.el6_4.noarch
  • candlepin-scl-quartz-2.1.5-5.el6_4.noarch
  • candlepin-scl-rhino-1.7R3-1.el6_4.noarch
  • candlepin-scl-runtime-1-5.el6_4.noarch
  • candlepin-selinux-0.9.23-1.el6_5.noarch
  • candlepin-tomcat6-0.9.23-1.el6_5.noarch
  • elasticsearch-0.90.10-6.el6sat.noarch
  • foreman-1.6.0.44-1.el6sat.noarch
  • foreman-compute-1.6.0.44-1.el6sat.noarch
  • foreman-gce-1.6.0.44-1.el6sat.noarch
  • foreman-libvirt-1.6.0.44-1.el6sat.noarch
  • foreman-ovirt-1.6.0.44-1.el6sat.noarch
  • foreman-postgresql-1.6.0.44-1.el6sat.noarch
  • foreman-proxy-1.6.0.30-1.el6sat.noarch
  • foreman-selinux-1.6.0.14-1.el6sat.noarch
  • foreman-vmware-1.6.0.44-1.el6sat.noarch
  • katello-1.5.0-30.el6sat.noarch
  • katello-certs-tools-1.5.6-1.el6sat.noarch
  • katello-default-ca-1.0-1.noarch
  • katello-installer-0.0.64-1.el6sat.noarch
  • katello-server-ca-1.0-1.noarch
  • openldap-2.4.23-32.el6_4.1.x86_64
  • pulp-katello-0.3-4.el6sat.noarch
  • pulp-nodes-common-2.4.1-0.7.beta.el6sat.noarch
  • pulp-nodes-parent-2.4.1-0.7.beta.el6sat.noarch
  • pulp-puppet-plugins-2.4.1-0.7.beta.el6sat.noarch
  • pulp-puppet-tools-2.4.1-0.7.beta.el6sat.noarch
  • pulp-rpm-plugins-2.4.1-0.7.beta.el6sat.noarch
  • pulp-selinux-2.4.1-0.7.beta.el6sat.noarch
  • pulp-server-2.4.1-0.7.beta.el6sat.noarch
  • python-ldap-2.3.10-1.el6.x86_64
  • ruby193-rubygem-net-ldap-0.3.1-3.el6sat.noarch
  • ruby193-rubygem-runcible-1.1.0-2.el6sat.noarc

How reproducible:
Always.

Steps to Reproduce:
1. Issue an HTTP GET request to api/v2/auth_source_ldaps/:id.
2. Examine the response. Look for a "host" parameter.

Actual results:
The following parameters are returned when reading an auth source LDAP:

[
u'account',
u'name',
u'attr_firstname',
u'attr_lastname',
u'attr_login',
u'attr_mail',
u'attr_photo',
u'base_dn',
u'created_at',
u'id',
u'ldap_filter',
u'onthefly_register',
u'port',
u'tls',
u'type',
u'updated_at',
]

Expected results:
Same as above, but with 'host' in the list too.

Additional info:
n/a

Associated revisions

Revision d6b1f7b4 (diff)
Added by Joseph Magen almost 4 years ago

fixes #7409 - API v2 - add host attribute to GET response for auth source ldap

Revision fab20708 (diff)
Added by Joseph Magen almost 4 years ago

fixes #7409 - API v2 - add host attribute to GET response for auth source ldap

(cherry picked from commit d6b1f7b47c63052d3579720f5aa307d65d24b3b5)

History

#1 Updated by Dominic Cleal almost 4 years ago

  • Category set to API
  • Assignee deleted (Bryan Kearney)

#2 Updated by Joseph Magen almost 4 years ago

  • Status changed from New to Assigned
  • Assignee set to Joseph Magen
  • Target version set to 1.7.3

#3 Updated by The Foreman Bot almost 4 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/1764 added
  • Pull request deleted ()

#4 Updated by Joseph Magen almost 4 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#5 Updated by Dominic Cleal almost 4 years ago

  • Legacy Backlogs Release (now unused) set to 22

Also available in: Atom PDF