Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Re-evaluate code coverage of tests #21

Open
j3parker opened this issue Feb 14, 2016 · 1 comment
Open

Re-evaluate code coverage of tests #21

j3parker opened this issue Feb 14, 2016 · 1 comment
Labels
needs-investigation semver-neutral
Milestone
6.0.0

Comments

@j3parker
Copy link
Member

No description provided.

@j3parker j3parker added this to the 5.0.0 milestone Feb 14, 2016
j3parker added a commit that referenced this issue Feb 14, 2016
@j3parker
Merge pull request #21 in AUTH/d2l.security.oauth2.webapi from ~JPARK…
0716ee2 
…ER/d2l.security.oauth2.webapi:add-authorization-attributes to master

* commit '7a509bc1ecc2f07efe01c7af572924b28848f57b':
  Improve error message
  Use AllowMultiple = false for attributes and remove extension method
  Switch to explicitly requiring developers to specify allowed principal types and combine all of those attributes into the single AllowFromAttribute
  Add UsersOnlyAttribute
  Require actions to specify a scope explicitly - throw from DefaultAuthorizationAttribute if they don't (programmer error)
  Rename DefaultStrictAuthorizationAttribute to DefaultAuthorizationAttribute (strictness is going to shrink soon)
  Explicitly don't follow inheritance chain in GetSingleAttribute
  AllowUsers -> AllowUsersAndServices, make it check the PrincipalType (now that we are allowing anon principals through authentication)
  Add integration tests for DefaultStrictAuthorizationAttribute
  Fix bug in ServicesOnlyAttribute
  Move minimal "RunTest" helper into TestUtilities
  Move stuff from SetUpFixture into TestUtilities
  Add authorization attributes
@j3parker j3parker removed this from the 5.0.0 milestone Feb 14, 2016
@j3parker j3parker added this to the 5.0.0 milestone Feb 14, 2016
@j3parker j3parker removed this from the 5.0.0 milestone Mar 3, 2016
@j3parker
Copy link
Member Author

j3parker commented Mar 3, 2016

"j3parker removed this from the 5.0.0 milestone just now"

heh heh heh

@j3parker j3parker added this to the 6.0.0 milestone May 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
needs-investigation semver-neutral
Projects
None yet
Milestone
6.0.0
Development

No branches or pull requests
1 participant
@j3parker