commit270739a45aAuthor: Jens Langhammer <jens.langhammer@beryju.org> Date: Thu May 28 21:50:43 2020 +0200 admin: fix policy testing form not showing the correct result commitdf8995deedAuthor: Jens L <jens@beryju.org> Date: Thu May 28 21:45:54 2020 +0200 policies/*: remove Policy.negate, order, timeout (#39) policies: rewrite engine to use PolicyBinding for order/negate/timeout policies: rewrite engine to use PolicyResult instead of tuple commitfdfc6472d2Author: Jens Langhammer <jens.langhammer@beryju.org> Date: Thu May 28 10:36:10 2020 +0200 admin: fixup some urls commitbc495828e7Author: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com> Date: Thu May 28 09:39:28 2020 +0200 build(deps): bump django-redis from 4.11.0 to 4.12.1 (#38) Bumps [django-redis](https://github.com/jazzband/django-redis) from 4.11.0 to 4.12.1. - [Release notes](https://github.com/jazzband/django-redis/releases) - [Changelog](https://github.com/jazzband/django-redis/blob/master/CHANGES.rst) - [Commits](https://github.com/jazzband/django-redis/compare/4.11.0...4.12.1) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com> commitfa138a273fAuthor: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com> Date: Thu May 28 08:59:19 2020 +0200 build(deps): bump boto3 from 1.13.17 to 1.13.18 (#37) Bumps [boto3](https://github.com/boto/boto3) from 1.13.17 to 1.13.18. - [Release notes](https://github.com/boto/boto3/releases) - [Changelog](https://github.com/boto/boto3/blob/develop/CHANGELOG.rst) - [Commits](https://github.com/boto/boto3/compare/1.13.17...1.13.18) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
		
			
				
	
	
		
			65 lines
		
	
	
		
			2.3 KiB
		
	
	
	
		
			Python
		
	
	
	
	
	
			
		
		
	
	
			65 lines
		
	
	
		
			2.3 KiB
		
	
	
	
		
			Python
		
	
	
	
	
	
"""OIDC Permission checking"""
 | 
						|
from typing import Optional
 | 
						|
 | 
						|
from django.contrib import messages
 | 
						|
from django.db.models.deletion import Collector
 | 
						|
from django.http import HttpRequest, HttpResponse
 | 
						|
from django.shortcuts import redirect
 | 
						|
from oidc_provider.models import Client
 | 
						|
from structlog import get_logger
 | 
						|
 | 
						|
from passbook.audit.models import Event, EventAction
 | 
						|
from passbook.core.models import Application, Provider, User
 | 
						|
from passbook.policies.engine import PolicyEngine
 | 
						|
 | 
						|
LOGGER = get_logger()
 | 
						|
 | 
						|
 | 
						|
def client_related_provider(client: Client) -> Optional[Provider]:
 | 
						|
    """Lookup related Application from Client"""
 | 
						|
    # because oidc_provider is also used by app_gw, we can't be
 | 
						|
    # sure an OpenIDProvider instance exists. hence we look through all related models
 | 
						|
    # and choose the one that inherits from Provider, which is guaranteed to
 | 
						|
    # have the application property
 | 
						|
    collector = Collector(using="default")
 | 
						|
    collector.collect([client])
 | 
						|
    for _, related in collector.data.items():
 | 
						|
        related_object = next(iter(related))
 | 
						|
        if isinstance(related_object, Provider):
 | 
						|
            return related_object
 | 
						|
    return None
 | 
						|
 | 
						|
 | 
						|
def check_permissions(
 | 
						|
    request: HttpRequest, user: User, client: Client
 | 
						|
) -> Optional[HttpResponse]:
 | 
						|
    """Check permissions, used for
 | 
						|
    https://django-oidc-provider.readthedocs.io/en/latest/
 | 
						|
    sections/settings.html#oidc-after-userlogin-hook"""
 | 
						|
    provider = client_related_provider(client)
 | 
						|
    if not provider:
 | 
						|
        return redirect("passbook_providers_oauth:oauth2-permission-denied")
 | 
						|
    try:
 | 
						|
        application = provider.application
 | 
						|
    except Application.DoesNotExist:
 | 
						|
        return redirect("passbook_providers_oauth:oauth2-permission-denied")
 | 
						|
    LOGGER.debug(
 | 
						|
        "Checking permissions for application", user=user, application=application
 | 
						|
    )
 | 
						|
    policy_engine = PolicyEngine(application.policies.all(), user, request)
 | 
						|
    policy_engine.build()
 | 
						|
 | 
						|
    # Check permissions
 | 
						|
    result = policy_engine.result
 | 
						|
    if not result.passing:
 | 
						|
        for policy_message in result.messages:
 | 
						|
            messages.error(request, policy_message)
 | 
						|
        return redirect("passbook_providers_oauth:oauth2-permission-denied")
 | 
						|
 | 
						|
    Event.new(
 | 
						|
        EventAction.AUTHORIZE_APPLICATION,
 | 
						|
        authorized_application=application,
 | 
						|
        skipped_authorization=False,
 | 
						|
    ).from_http(request)
 | 
						|
    return None
 |