Bug #15286

Missing API for external auth sources

Added by Bryan Kearney about 2 years ago. Updated 8 days ago.

Status:Closed
Priority:Normal
Assignee:Dominik Hlavac Duran
Category:API
Target version:1.18.0
Difficulty: Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link:1336236 Found in Releases:
Pull request:https://github.com/theforeman/foreman/pull/4865, https://github.com/theforeman/foreman/pull/5352

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1336236
Description of problem:
hammer user-group external create requires auth_source_id which is not available when configuring IPA/IdM integration with
katello-installer --foreman-ipa-authentication=true

Version-Release number of selected component (if applicable):
6.1.8

How reproducible:
Always

Steps to Reproduce:
1. ipa-client-install
2. katello-installer --foreman-ipa-authentication=true
3. hammer user-group create --name=dummy --role-ids=1,2,3,4
User group [dummy] created
4. hammer user-group external create --name=dummy --user-group=dummy

Actual results:

Could not create external user group:
Missing arguments for 'auth_source_id'

--- No value of auth_source_id works

Expected results:

External group [dummy] created

Additional info:

[root@sat6 /]# hammer auth-source ldap list
---|------|--------|------|------------
ID | NAME | LDAPS? | PORT | SERVER TYPE
---|------|--------|------|------------


Related issues

Related to Foreman - Bug #19064: External group mapping cannot be set when --foreman-ipa-a... Closed 03/29/2017
Duplicated by Foreman - Feature #18584: Api for listing all auth sources Duplicate 02/21/2017
Blocks Hammer CLI - Feature #4356: Improve ways of passing auth source when creating a user Duplicate 02/17/2014
Blocks Hammer CLI - Feature #18805: Creating external user-group does not have any option --a... Duplicate 03/06/2017
Blocks Hammer CLI - Feature #19651: Command for listing all auth sources Closed 05/24/2017

Associated revisions

Revision 67b925e9
Added by Dominik Hlavac Duran 4 months ago

Fixes #15286 - New API for auth sources (#4865)

Revision cd43517a
Added by Michael Moll 4 months ago

Refs #15286 - fix Layout/IndentationWidth cop

[ci skip]

History

#1 Updated by Dominic Cleal about 2 years ago

  • Category set to Authorization
  • Status changed from New to Feedback

requires auth_source_id which is not available when configuring IPA/IdM integration [..]

An auth source is created when configuring authorize_login_delegation_auth_source_user_autocreate and a user logs in with the external mechanism. The auth source can then be used to create external user groups.

#2 Updated by Marek Hulán over 1 year ago

  • Status changed from Feedback to New
  • Target version set to 115

We should probably seed the external auth source, so the very first login attempt could already have some mapped external user groups. I hope that's acceptable fix, so reopening.

#3 Updated by Marek Hulán over 1 year ago

  • Related to Bug #19064: External group mapping cannot be set when --foreman-ipa-authentication=true is used and no EXTERNAL user has been created yet added

#4 Updated by Marek Hulán over 1 year ago

  • Status changed from New to Duplicate

Closing as a duplicate since after the external auth source is seeded (#19064) the hammer command works just fine.

#5 Updated by Marek Hulán over 1 year ago

  • Subject changed from hammer can't create external groups based on IPA/IdM integration to Missing API for external auth sources
  • Status changed from Duplicate to New

Oh no, that's actually not a duplicate, the problem reported here is that hammer can't list external auth source so user can't find out the id to use, I'll change the subject so it's more clear. And of course hammer needs API, our API only provides access to LDAP auth sources. So reopening.

#6 Updated by Dominic Cleal over 1 year ago

  • Category changed from Authorization to API

#7 Updated by Joseph Magen about 1 year ago

  • Assignee set to Joseph Magen

#8 Updated by Marek Hulán about 1 year ago

  • Target version changed from 115 to 1.13.4

#9 Updated by Marek Hulán about 1 year ago

  • Target version changed from 1.13.4 to 1.14.0

#10 Updated by Marek Hulán about 1 year ago

  • Target version changed from 1.14.0 to 1.14.3

#11 Updated by Daniel Lobato Garcia about 1 year ago

  • Duplicated by Feature #18584: Api for listing all auth sources added

#12 Updated by Dominik Hlavac Duran about 1 year ago

  • Blocks Feature #4356: Improve ways of passing auth source when creating a user added

#13 Updated by Dominik Hlavac Duran about 1 year ago

  • Blocks Feature #18805: Creating external user-group does not have any option --auth-source as a input parameter. added

#14 Updated by Dominik Hlavac Duran about 1 year ago

#15 Updated by Dominik Hlavac Duran about 1 year ago

  • Status changed from New to Assigned
  • Assignee changed from Joseph Magen to Dominik Hlavac Duran

#16 Updated by Marek Hulán 12 months ago

  • Target version changed from 1.14.3 to 1.17.0-RC2

#17 Updated by Marek Hulán 11 months ago

  • Target version changed from 1.17.0-RC2 to 1.18.0-RC2

#18 Updated by The Foreman Bot 11 months ago

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

#19 Updated by Marek Hulán 11 months ago

  • Target version changed from 1.18.0-RC2 to 214

#20 Updated by Marek Hulán 10 months ago

  • Target version changed from 214 to 1.16.0-RC2

#21 Updated by The Foreman Bot 10 months ago

  • Pull request https://github.com/theforeman/foreman/pull/4865 added

#22 Updated by Dominik Hlavac Duran 10 months ago

  • Pull request deleted (https://github.com/theforeman/foreman/pull/4727)

#23 Updated by Marek Hulán 9 months ago

  • Target version changed from 1.16.0-RC2 to 1.16.0-RC1

#24 Updated by Marek Hulán 9 months ago

  • Target version changed from 1.16.0-RC1 to 1.16.2

#25 Updated by Marek Hulán 8 months ago

  • Target version changed from 1.16.2 to 1.16.1

#26 Updated by Marek Hulán 7 months ago

  • Target version changed from 1.16.1 to 238

#27 Updated by Marek Hulán 4 months ago

  • Status changed from Ready For Testing to Closed

#28 Updated by Tomer Brisker 4 months ago

  • Legacy Backlogs Release (now unused) set to 330
  • Pull request https://github.com/theforeman/foreman/pull/5352 added

Also available in: Atom PDF