Project

General

Profile

Feature #8016

Ability to use tokenized authentication to hammer in lieu of username/password in configuration file.

Added by Dominic Cleal over 5 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Normal
Category:
Hammer core
Team Backlog:
Fixed in Releases:
Found in Releases:
In Kanboard:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1154382
Description of problem:
This RFE requests the ability to configure hammer with either

  • a token for login that is separate from the username/pass OR
  • a session based credential cache so that the user can be prompted for login once at the beginning of their CLI session.

The goal is to keep the user from placing an administrative username/password within hammer's configuration file, in cleartext.

Version-Release number of selected component (if applicable):
rubygem-hammer_cli-0.1.1-12.el6sat.noarch

How reproducible:

Steps to Reproduce:
1.
2.
3.

Actual results:

Expected results:

Additional info:


Related issues

Related to Foreman - Feature #1301: Consider adding a per-user API keyClosed
Related to Hammer CLI - Feature #13602: OAuth Support for HammerClosed
Related to Foreman - Feature #17487: Allow sessions for API callsClosed2016-11-25
Related to Hammer CLI CSV - Bug #17624: session api breaks hammer csvClosed2016-12-10
Related to Hammer CLI - Bug #17650: Basic authentication overrides any error with "Invalid username or password"Closed2016-12-13
Has duplicate Hammer CLI - Feature #8888: hammer auth should support token based authenticationClosed
Blocks Hammer CLI - Feature #8923: Ability to use Negotiate/Kerberos authentication to API and hammerNew

Associated revisions

Revision 49b9d453 (diff)
Added by Tomas Strachota over 3 years ago

Fixes #8016 - api connection moved to context (#227)

- connection moved to context
- connection is now instance, not class
- dependency injection for connection and connectors

Revision ce9ede29 (diff)
Added by Tomas Strachota over 3 years ago

Fixes #8016 - Session auth in hammer (#269)

- api connection moved to a separate object
- credentials moved to InteractiveBasicAuth
- new SessionAuthenticatorWrapper for handling api sessions
- sessions turned off by default

History

#1 Updated by Dominic Cleal over 5 years ago

  • Related to Feature #1301: Consider adding a per-user API key added

#2 Updated by Dominic Cleal over 5 years ago

  • Category set to Hammer core

#3 Updated by Tomáš Strachota over 5 years ago

  • Has duplicate Feature #8888: hammer auth should support token based authentication added

#4 Updated by Jan Pazdziora over 5 years ago

  • Blocks Feature #8923: Ability to use Negotiate/Kerberos authentication to API and hammer added

#5 Updated by Dominic Cleal over 4 years ago

#6 Updated by Tomáš Strachota almost 4 years ago

  • Target version set to 115

#7 Updated by Marek Hulán almost 4 years ago

  • Assignee set to Tomáš Strachota
  • Target version changed from 115 to 1.5.0

as we discussed yesterday, storing session ID might be enough, hammer would fallback to username/password credentials if session expired or no session was stored

#8 Updated by Tomáš Strachota over 3 years ago

  • Status changed from New to Assigned

#9 Updated by Marek Hulán over 3 years ago

  • Target version changed from 1.5.0 to 1.4.2

#10 Updated by Marek Hulán over 3 years ago

  • Target version changed from 1.4.2 to 1.4.4

#11 Updated by The Foreman Bot over 3 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/hammer-cli/pull/227 added

#12 Updated by The Foreman Bot over 3 years ago

  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/269 added

#13 Updated by Tomáš Strachota over 3 years ago

#14 Updated by Marek Hulán over 3 years ago

  • Target version changed from 1.4.4 to 1.10.1

#15 Updated by Anonymous over 3 years ago

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

#16 Updated by Thomas McKay over 3 years ago

  • Related to Bug #17624: session api breaks hammer csv added

#17 Updated by Tomáš Strachota over 3 years ago

  • Related to Bug #17650: Basic authentication overrides any error with "Invalid username or password" added

#18 Updated by Tomáš Strachota over 3 years ago

  • Legacy Backlogs Release (now unused) set to 235

#19 Updated by Tomáš Strachota over 3 years ago

  • Legacy Backlogs Release (now unused) changed from 235 to 236

Also available in: Atom PDF