Project

General

Profile

Bug #33147

Ansible collection repo validate both auth url and token are supplied

Added by Samir Jha 10 months ago. Updated 10 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Repositories
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1984890

Description of problem:
Ansible collection repo fails to sync from the ansible hub, both token-based auth and SSO URL fail

Version-Release number of selected component (if applicable):
Satellite 6.10 sn 10

How reproducible:
always

Steps to Reproduce:
1. create ansible collection repo:

Upstream url:
https://cloud.redhat.com/api/automation-hub/

requirements.yml:
---
collections:
- redhat.satellite_operations

either use
Auth URL: https://sso.redhat.com/auth/realms/redhat-external/protocol/openid-connect/token

or generated Auth token (both based on https://cloud.redhat.com/ansible/automation-hub/token/)

2. sync repo

Actual results:

Synchronization tasks fail,
if using auth url, the error is:

400, message='Bad Request', url=URL

if using token, the the error is:

401, message='Unauthorized', url=URL

Expected results:
Successful sync

Additional info:
Looking directly at https://sso.redhat.com/auth/realms/redhat-external/protocol/openid-connect/token I'm seeing "An internal server error has occurred", the issue may as well be at ansible-hub

Associated revisions

Revision eed98173 (diff)
Added by Samir Jha 10 months ago

Fixes #33147 - Add validation around Ansible auth url and token (#9500)

History

#1 Updated by The Foreman Bot 10 months ago

  • Assignee set to Samir Jha
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/9500 added

#2 Updated by Ian Ballou 10 months ago

  • Triaged changed from No to Yes
  • Target version set to Katello 4.2.0

#3 Updated by The Foreman Bot 10 months ago

  • Fixed in Releases Katello 4.3.0 added

#4 Updated by Samir Jha 10 months ago

  • Status changed from Ready For Testing to Closed

#5 Updated by The Foreman Bot 10 months ago

  • Pull request https://github.com/Katello/katello/pull/9529 added

#6 Updated by Samir Jha 10 months ago

  • Pull request deleted (https://github.com/Katello/katello/pull/9529)

Also available in: Atom PDF