search

anvilresearch / connect

A modern authorization server built to authenticate your users and protect your APIs
http://anvil.io
MIT License
361 stars 85 forks source link

fix(oidc): ensure role present and has name attr #339

Closed tomkersten closed 8 years ago

tomkersten commented 8 years ago

In some cases, I was getting a null value back for a role returned in the listByUsers callback.

I am working on figuring out what the issue in the underlying database is, but this guards against it in the event that it occurs.

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 79.743% when pulling 342093d25358231edc5c4cff2adf34ab55ef7340 on tomkersten-requireVerifiedEmail into f8752f1447a705866d4cb86f6cfa0a478d6a2635 on master.

christiansmith commented 8 years ago

Thanks!