fastapi
fastapi copied to clipboard
Add support to Client Credentials Flow (OAuth2)
issue : https://github.com/tiangolo/fastapi/issues/774
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
0bb8920
) 100.00% compared to head (7090cdb
) 100.00%. Report is 1198 commits behind head on master.
Additional details and impacted files
@@ Coverage Diff @@
## master #5052 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 535 537 +2
Lines 13825 13963 +138
==========================================
+ Hits 13825 13963 +138
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
📝 Docs preview for commit 48611ac42fb5581c725341376acad67b3258ac1d at: https://62adefeee6a1801161daafce--fastapi.netlify.app
All for this! I do see you didn't provide any tests though, is that on the planning?
Will add test yep, sorry!
📝 Docs preview for commit 6b5a4cc58aba78bd5edc28e20aaea7b1d55599f9 at: https://62b2119df868f015e2b47298--fastapi.netlify.app
📝 Docs preview for commit b07569d1708eff6fb5aacb05aab023d64c23a719 at: https://62b7196e1aa56f7e7005c960--fastapi.netlify.app
📝 Docs preview for commit 9c3dda986c87e3e012072483e4dec85a441f8e03 at: https://62b71fa51aa56f036105c86c--fastapi.netlify.app
📝 Docs preview for commit 232394d001e851cb52b64e0407e990ce69ba3be1 at: https://62b720b7f57baf090dfb812e--fastapi.netlify.app
Ready to go!
📝 Docs preview for commit cb0305e1ab6f000aa5f0d7a1c9fa73629ed31c7c at: https://62b722f3e2b0760a373ddeb6--fastapi.netlify.app
📝 Docs preview for commit f02c4aaf1f0a256607c9302ef0891ec400584946 at: https://62d18ce54624d43f5615452b--fastapi.netlify.app
📝 Docs preview for commit a1893f053b9a4084cc134b49e0a6744755420a44 at: https://62e29b96e3bee326f645bf5b--fastapi.netlify.app
📝 Docs preview for commit 7090cdbf9312b61f0e01445436d08f2fee2bc9b7 at: https://630e1bf89f0bdf6afee85fc5--fastapi.netlify.app
Hey @JarroVGIT, just wondering, is there a chance to get this merged? I'm willing to pick this up if there's any additional work needed to be done before merging.
@kokolem I am not the maintainer of this project; I just do some reviews now and then and help people out with issues. @tiangolo is the one that determines what is merged and not, but as you can see there is quite a backlog he is going through. Eventually he'll get to this, I'm sure :)
could some docs be added to the PR?
Would it be appropriate to add some info about using this flow on the advanced/security/oauth2-scopes/ docs page?
@tiangolo is there any plans to implement client credentials flow?