Project

General

Profile

Bug #49574

mgr/dashboard: ERROR: test_a_set_login_credentials (tasks.mgr.dashboard.test_auth.AuthTest)

Added by Ernesto Puerta about 2 months ago. Updated 6 days ago.

Status:
Resolved
Priority:
High
Category:
dashboard/auth-sso
Target version:
% Done:

0%

Source:
Q/A
Tags:
dependency
Backport:
pacific,octopus,nautilus
Regression:
No
Severity:
2 - major
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Description of problem

INFO:tasks.cephfs_test_runner:======================================================================
INFO:tasks.cephfs_test_runner:ERROR: test_a_set_login_credentials (tasks.mgr.dashboard.test_auth.AuthTest)
INFO:tasks.cephfs_test_runner:----------------------------------------------------------------------
INFO:tasks.cephfs_test_runner:Traceback (most recent call last):                                                                                                                                                       
INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/git.ceph.com_ceph-c_18af150d0d7db4092e9941964be7efd7560b0c4b/qa/tasks/mgr/dashboard/test_auth.py", line 44, in test_a_set_login_credentials
INFO:tasks.cephfs_test_runner:    self._validate_jwt_token(data['token'], "admin2", data['permissions'])
INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/git.ceph.com_ceph-c_18af150d0d7db4092e9941964be7efd7560b0c4b/qa/tasks/mgr/dashboard/test_auth.py", line 23, in _validate_jwt_token
INFO:tasks.cephfs_test_runner:    payload = jwt.decode(token, verify=False)  
INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/git.ceph.com_git_teuthology_3082387bba74fcd24c9700593d10418152d53c97/virtualenv/lib/python3.6/site-packages/jwt/api_jwt.py", line 113, in decode
INFO:tasks.cephfs_test_runner:    decoded = self.decode_complete(jwt, key, algorithms, options, **kwargs)
INFO:tasks.cephfs_test_runner:  File "/home/teuthworker/src/git.ceph.com_git_teuthology_3082387bba74fcd24c9700593d10418152d53c97/virtualenv/lib/python3.6/site-packages/jwt/api_jwt.py", line 80, in decode_complete
INFO:tasks.cephfs_test_runner:    'It is required that you pass in a value for the "algorithms" argument when calling decode().'
INFO:tasks.cephfs_test_runner:jwt.exceptions.DecodeError: It is required that you pass in a value for the "algorithms" argument when calling decode().

Environment

https://pulpito.ceph.com/sage-2021-03-01_20:24:37-rados-wip-sage-testing-2021-03-01-1118-distro-basic-smithi/

How reproducible

Teuthology runs or pyjwt >2.0.0

Additional info

The interface for the jwt.decode() has changed and the verify option is dropped (it was dropped long time ago, but now it's checked):

https://github.com/jpadilla/pyjwt/blame/ae080f472c913ad94456fd9e10b05ec2d038b7cc/jwt/api_jwt.py#L71

https://github.com/jpadilla/pyjwt/issues/594


Related issues

Copied to mgr - Backport #49594: octopus: mgr/dashboard: ERROR: test_a_set_login_credentials (tasks.mgr.dashboard.test_auth.AuthTest) Resolved
Copied to mgr - Backport #49595: pacific: mgr/dashboard: ERROR: test_a_set_login_credentials (tasks.mgr.dashboard.test_auth.AuthTest) Resolved
Copied to mgr - Backport #49596: nautilus: mgr/dashboard: ERROR: test_a_set_login_credentials (tasks.mgr.dashboard.test_auth.AuthTest) Resolved

History

#2 Updated by Alfonso Martínez about 2 months ago

  • Status changed from In Progress to Fix Under Review
  • Pull request ID set to 39801

#3 Updated by Ernesto Puerta about 2 months ago

  • Status changed from Fix Under Review to Pending Backport

#4 Updated by Backport Bot about 2 months ago

  • Copied to Backport #49594: octopus: mgr/dashboard: ERROR: test_a_set_login_credentials (tasks.mgr.dashboard.test_auth.AuthTest) added

#5 Updated by Backport Bot about 2 months ago

  • Copied to Backport #49595: pacific: mgr/dashboard: ERROR: test_a_set_login_credentials (tasks.mgr.dashboard.test_auth.AuthTest) added

#6 Updated by Backport Bot about 2 months ago

  • Copied to Backport #49596: nautilus: mgr/dashboard: ERROR: test_a_set_login_credentials (tasks.mgr.dashboard.test_auth.AuthTest) added

#7 Updated by Ernesto Puerta about 2 months ago

  • Status changed from Pending Backport to Resolved

#8 Updated by Ernesto Puerta 6 days ago

  • Project changed from mgr to Dashboard
  • Category changed from dashboard/auth-sso to dashboard/auth-sso

Also available in: Atom PDF