Merge pull request #135 from britive/v3.2.0-alpha.5

V3.2.0 alpha.5

Author: theborch

CHANGELOG.md

@@ -1,6 +1,6 @@
 # Change Log (v2.8.1+)
 
-## v3.2.0-alpha.* [2024-12-23]
+## v3.2.0-alpha.* [2025-01-07]
 
 __What's New:__
 
@@ -17,10 +17,10 @@ __Enhancements:__
 * Added `add_favorite` and `delete_favorite` to `my_resources`.
 * Added checkout approvals to `my_resources`.
 * Added ITSM to checkout approvals.
-* Added `include_approval_status` to `my_access.list_profiles`.
 * Added `(create|list|update|delete)_filter`) to `my_access`.
 * Added `response_templates` functionality for `access_broker` credentials.
 * Added `request_approval[_by_name]|withdraw_approval_request[_by_name]` to `my_resources`.
+* Added `my_access.list` to retrieve access details with new `type=sdk` option.
 
 __Bug Fixes:__
 

src/britive/__init__.py

@@ -1 +1 @@
-__version__ = '3.2.0-alpha.4'
+__version__ = '3.2.0-alpha.5'

src/britive/access_broker/__init__.py

@@ -1,13 +1,13 @@
 from .brokers import Brokers
 from .pools import Pools
-from .profiles.profiles import Profile
-from .resources.resources import Resources
+from .profiles import Profiles
+from .resources import Resources
 from .response_templates import ResponseTemplates
 
 
 class AccessBroker:
     def __init__(self, britive) -> None:
-        self.profiles = Profile(britive)
+        self.profiles = Profiles(britive)
         self.resources = Resources(britive)
         self.response_templates = ResponseTemplates(britive)
         self.brokers = Brokers(britive)

…itive/access_broker/profiles/profiles.py …itive/access_broker/profiles/__init__.pysrc/britive/access_broker/profiles/profiles.py renamed to src/britive/access_broker/profiles/__init__.py src/britive/access_broker/profiles/profiles.py renamed to src/britive/access_broker/profiles/__init__.py

@@ -2,7 +2,7 @@
 from .policies import Policies
 
 
-class Profile:
+class Profiles:
     def __init__(self, britive) -> None:
         self.britive = britive
         self.base_url = f'{self.britive.base_url}/resource-manager/profiles'

src/britive/access_broker/profiles/policies.py

@@ -82,7 +82,7 @@ def get(self, profile_id: str, policy_id: str) -> dict:
 
         return self.britive.get(f'{self.base_url}/{profile_id}/policies/{policy_id}')
 
-    def update(  # noqa: PLR0913
+    def update(
         self,
         profile_id: str,
         policy_id: str,

…ive/access_broker/resources/resources.py …tive/access_broker/resources/__init__.pysrc/britive/access_broker/resources/resources.py renamed to src/britive/access_broker/resources/__init__.py src/britive/access_broker/resources/resources.py renamed to src/britive/access_broker/resources/__init__.py

@@ -107,8 +107,7 @@ def map(
         if map_user_to_account_in_all_application_environments:
             url += f'/{user_id}'
             return self.britive.post(url, json={'saveToAllEnvs': True})
-        else:
-            return self.britive.post(url, json=[user_id])
+        return self.britive.post(url, json=[user_id])
 
     def unmap(
         self,

src/britive/application_management/accounts.py

@@ -1,7 +1,7 @@
 import json
 from typing import Union
 
-from .. import exceptions
+from britive import exceptions
 
 creation_defaults = {
     'expirationDuration': 3600000,
@@ -127,7 +127,7 @@ def get(self, application_id: str, profile_id: str, summary: bool = None) -> dic
             for profile in self.list(application_id=application_id):
                 if profile['papId'] == profile_id:
                     return profile
-            raise exceptions.ProfileNotFound()
+            raise exceptions.ProfileNotFound
         params = {}
         if summary:
             params['view'] = 'summary'
@@ -382,7 +382,7 @@ def lint_condition(
         :returns: Results of the lint operation.
         """
 
-        url = f'{self.base_url}/{profile_id}/permissions/{permission_name}/' f'{permission_type}/constraints/condition'
+        url = f'{self.base_url}/{profile_id}/permissions/{permission_name}/{permission_type}/constraints/condition'
 
         params = {'operation': 'validate'}
 

src/britive/application_management/profiles.py

@@ -23,8 +23,7 @@ def scan(self, application_id: str, environment_id: str = None) -> dict:
 
         if environment_id:
             return self.britive.post(f'{self.base_url}/{application_id}/environments/{environment_id}/scans')
-        else:
-            return self.britive.post(f'{self.base_url}/{application_id}/scan')
+        return self.britive.post(f'{self.base_url}/{application_id}/scan')
 
     def status(self, task_id: str) -> dict:
         """

src/britive/application_management/scans.py

@@ -58,8 +58,8 @@ def query(
             raise ValueError('from_time must occur before to_time.')
 
         params = {
-            'from': from_time.isoformat(sep='T', timespec='seconds').split("+")[0] + 'Z',
-            'to': to_time.isoformat(sep='T', timespec='seconds').split("+")[0] + 'Z',
+            'from': from_time.isoformat(sep='T', timespec='seconds').split('+')[0] + 'Z',
+            'to': to_time.isoformat(sep='T', timespec='seconds').split('+')[0] + 'Z',
         }
         if filter_expression:
             params['filter'] = filter_expression