Remove IAccessTokenRetriever w/resource param

DuendeSoftware / BFF

Framework for ASP.NET Core to secure SPAs using the Backend-for-Frontend (BFF) pattern

Other

333 stars 75 forks source link

Remove IAccessTokenRetriever w/resource param #175

Closed josephdecock closed 1 year ago

josephdecock commented 1 year ago

Update to D.ATM 2.0.2, remove the IAccessTokenRetriever sample that set a resource param, and instead, set the resource param using BffUserAccessTokenParameters.

brockallen commented 1 year ago

oh jeeze -- you're way ahead of me. i missed this PR before i submitted this one: https://github.com/DuendeSoftware/BFF/pull/176

anyway -- we will review both

brockallen commented 1 year ago

I had a look. The two things that come to mind I'd like to have:

docs or samples to show how to enable more logging so a customer can understand why something might not be working
standalone sample for resource indicators