Skip to content
This repository has been archived by the owner on Apr 24, 2024. It is now read-only.

[WIP]Nesting of authorized directives #1113

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[WIP]Nesting of authorized directives #1113

wants to merge 1 commit into from

Conversation

axos88
Copy link

@axos88 axos88 commented May 28, 2016

Adding tests for the authorize directive.

Note that one use case fails currently, mentioned in #1112.

I tried modifying the authorize directive to:

    extract(check).flatMap[HNil](if (_) pass else reject(AuthorizationFailedRejection)) &
      cancelRejection(AuthorizationFailedRejection)

    extract(check).flatMap[HNil](if (_) pass else reject(AuthorizationFailedRejection)))

    extract(check).flatMap[HNil](if (_) pass & cancelRejection(AuthorizationFailedRejection) else reject(AuthorizationFailedRejection))

But it just moves around the failing use case.

Any insight is appreciated.

@axos88 axos88 changed the title PR for #1112 Nesting of authorized directives #1112 May 28, 2016
@axos88 axos88 changed the title Nesting of authorized directives #1112 Nesting of authorized directives May 28, 2016
@axos88 axos88 changed the title Nesting of authorized directives [WIP]Nesting of authorized directives May 28, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@axos88