Bug #7409

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

Added by Dominic Cleal about 3 years ago. Updated about 3 years ago.

Status:Closed
Priority:Normal
Assigned To:Joseph Magen
Category:API
Target version:Sprint 29
Difficulty: Bugzilla link:1140313
Found in release: Pull request:https://github.com/theforeman/foreman/pull/1764
Story points-
Velocity based estimate-
Release1.6.1Release relationshipAuto

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
Added by Joseph Magen about 3 years ago

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

Revision fab20708
Added by Joseph Magen about 3 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 about 3 years ago

  • Category set to API
  • Assigned To deleted (Bryan Kearney)

#2 Updated by Joseph Magen about 3 years ago

  • Status changed from New to Assigned
  • Assigned To set to Joseph Magen
  • Target version set to Sprint 29

#3 Updated by The Foreman Bot about 3 years ago

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

#4 Updated by Joseph Magen about 3 years ago

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

#5 Updated by Dominic Cleal about 3 years ago

  • Release set to 1.6.1

Also available in: Atom PDF