dash icon indicating copy to clipboard operation
dash copied to clipboard

Add support for allowing request_refresh_jwt hooks with json response

Open C0deBeez opened this issue 1 year ago • 2 comments

Contributor Checklist I have broken down my PR scope into the following TODO tasks

Implement request_refresh_jwt hooks Allow JSON format responses I have run the tests locally and they passed. (refer to testing section in contributing)

I have added tests, or extended existing tests, to cover any new features or bugs fixed in this PR

C0deBeez avatar Aug 06 '24 17:08 C0deBeez

hi @yzcheung - thanks very much for your PR. We're currently trying to figure out which bits of authentication should go in dash, which should go in dash-auth, and which we should reserve for our enterprise offering - I've assigned this to myself so I don't lose track of it, and I hope to have some answers for you in a couple or three weeks. cheers - @gvwilson

gvwilson avatar Aug 07 '24 13:08 gvwilson

hi @yzcheung - thanks very much for your PR. We're currently trying to figure out which bits of authentication should go in dash, which should go in dash-auth, and which we should reserve for our enterprise offering - I've assigned this to myself so I don't lose track of it, and I hope to have some answers for you in a couple or three weeks. cheers - @gvwilson

Cool, thanks for the update. Looking forward to hearing back soon!

C0deBeez avatar Aug 07 '24 14:08 C0deBeez