Skip to content

Commit 9f7c3b3

Browse files
AlanCodingdmzoneill
authored andcommitted
Fix linters and tests for new service_id key
1 parent 81369f4 commit 9f7c3b3

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

ansible_base/jwt_consumer/common/auth.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,10 @@ def parse_jwt_token(self, request):
115115
# Either the user wasn't cached or the requested user was not in the DB so we need to make a new one
116116
try:
117117
resource = Resource.create_resource(
118-
ResourceType.objects.get(name="shared.user"), resource_data=self.token["user_data"], ansible_id=self.token["sub"], service_id=self.token["service_id"]
118+
ResourceType.objects.get(name="shared.user"),
119+
resource_data=self.token["user_data"],
120+
ansible_id=self.token["sub"],
121+
service_id=self.token["service_id"],
119122
)
120123
self.user = resource.content_object
121124
logger.info(f"New user {self.user.username} created from JWT auth")

test_app/tests/conftest.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@
2626
from ansible_base.oauth2_provider.fixtures import * # noqa: F403, F401
2727
from ansible_base.rbac import permission_registry
2828
from ansible_base.rbac.models import RoleDefinition
29+
from ansible_base.resource_registry.models.service_identifier import service_id
2930
from test_app import models
3031

3132

@@ -517,6 +518,7 @@ def __init__(self):
517518
"exp": int(expiration_date.timestamp()),
518519
"aud": "ansible-services",
519520
"sub": "1e3de989-5286-48a6-83d4-5de9a6618ffd",
521+
"service_id": service_id(),
520522
"user_data": {
521523
"username": "john.westcott.iv",
522524
"first_name": "john",

0 commit comments

Comments
 (0)