search

anvilresearch / connect

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

UserInfo update #303

Closed christiansmith closed 8 years ago

christiansmith commented 8 years ago

This is not ready to merge. We need to review and ensure the endpoint is properly mapping from OIDC _-based attribute names to the internal camel case convention.

christiansmith commented 8 years ago

As I was working on the tests for this, it occurred to me again that we don't have a good way to delete properties, etc. On researching how other APIs deal with this sort of thing I discovered RFC6902. Before merging, I think we should consider this type of scheme for this endpoint and all the PATCH endpoints in the REST API.

christiansmith commented 8 years ago

I've created a new issue for RFC6902 JSON Patch API and will merge this now. Thanks @EternalDeiwos for getting the ball rolling on this requirement. Great work.