Skip to content
This repository was archived by the owner on Feb 25, 2019. It is now read-only.

Various bugfixes and style adjustments #110

Merged
merged 17 commits into from
Jun 30, 2015
Merged

Various bugfixes and style adjustments #110

merged 17 commits into from
Jun 30, 2015

Conversation

adalinesimonian
Copy link
Member

Pardon me if the commit messages aren't quite what they should be; I'm not yet accustomed to the style yet. Please let me know if anything is off kilter.

Test results:

  1018 passing (3s)
  33 pending
  1 failing

  1) Verify Scoped Client Registration with sufficient scope should not provide an error:
     AssertionError: expected { Object (name, error, ...) } to be undefined

@adalinesimonian
fix(boot): Define variables that are used but not defined
9553367
@adalinesimonian
fix(client): Fix callback function name
756a1b5
@adalinesimonian
fix(client): Remove unused variable
f89e8bd
@adalinesimonian
fix(openid): Correct use of undefined variable
269d524
@adalinesimonian
fix(register): Fix missing NotFoundError require
51e2796
@adalinesimonian
fix(userinfo): Add missing NotFoundError require
612f4e3
@adalinesimonian
fix(userapplications): Add missing semicolon
abd38c5
@adalinesimonian
style(server): Add missing semicolons.
572cbcf
@adalinesimonian
chore(signin): Remove comment which is no longer relevant
3d1ccca
@adalinesimonian
fix(authorizations): Use correct function to send only status code
3a27f8f
@adalinesimonian
refactor(templates): Remove unnecessary .gitkeep
6c6ec99
@adalinesimonian
refactor(oauth): Remove unused local variable
df080ad
@adalinesimonian
fix(signout): Update deprecated express API call
40ee238
@adalinesimonian
fix(time-utils): Fix variable double-definition
3886710
@adalinesimonian
style(multiple): Add missing semicolons
743899d
@adalinesimonian
chore(test): Update signout test to match express v4 API
2d40794
@adalinesimonian
fix(oidc): Don't dereference indexOf if scope is falsy
7b65497
christiansmith added a commit that referenced this pull request Jun 30, 2015
@christiansmith
Merge pull request #110 from vsimonian/master
ad794ab 
Various bugfixes and style adjustments
@christiansmith christiansmith merged commit ad794ab into anvilresearch:master Jun 30, 2015
@christiansmith
Copy link
Member

Great work! Thanks!

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.
2 participants
@adalinesimonian @christiansmith