added flag, whether or not a multi tenant app and added tests to test…#17
added flag, whether or not a multi tenant app and added tests to test…#17fingineering wants to merge 1 commit into425show:mainfrom
Conversation
|
I’m sorry but I don’t have access to this repo anymore :( |
|
What a pity, anyone else, who can help, or is this repo abandoned? |
|
@fingineering or @cmatskas since it is MIT licensed, any chance one of you are willing to be the maintainer and we fork it? Or do you recommend something else to use instead? |
|
@emirkmo unfortunately, as much as I would like to help, I’m not part of Microsoft and I don’t have a permission to manage this repo any more. At this point, IMHO your best bet would be to fork the repo |
|
Thanks, will do that. Didn't want to step on any toes. |
|
Pardon the late response, well, in principle I could try to maintain the repo and the package - but I'm not part of Microsoft as well. |
|
FTR I have forked the code into my codebase. Since I didn’t need all the features I stripped it down a bit. I could see myself open sourcing it but would much rather latch onto someone with my experience maintaining an open source python package. I do warn to alert anyone using this package that |
Changes
Added a feature flag, to enable authentication, when using a multitenant application
Added the parameter
multitenantto the list of global variables, which defaults to false. Therefore the existing behavior is preserved.In case of a
multitenant=Truethe issuer of the token is omited during the decoding.Tests
Added two tests:
enabled the setting of CLIENT_ID, TENANT_ID and USER_TOKEN via environment variables in a way that the original behavior is preserved