fastapi icon indicating copy to clipboard operation
fastapi copied to clipboard

Add support to Client Credentials Flow (OAuth2)

Open Riuzaky77 opened this issue 2 years ago • 18 comments

issue : https://github.com/tiangolo/fastapi/issues/774

Riuzaky77 avatar Jun 18 '22 15:06 Riuzaky77

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.

codecov[bot] avatar Jun 18 '22 15:06 codecov[bot]

📝 Docs preview for commit 48611ac42fb5581c725341376acad67b3258ac1d at: https://62adefeee6a1801161daafce--fastapi.netlify.app

github-actions[bot] avatar Jun 18 '22 15:06 github-actions[bot]

All for this! I do see you didn't provide any tests though, is that on the planning?

Will add test yep, sorry!

Riuzaky77 avatar Jun 20 '22 14:06 Riuzaky77

📝 Docs preview for commit 6b5a4cc58aba78bd5edc28e20aaea7b1d55599f9 at: https://62b2119df868f015e2b47298--fastapi.netlify.app

github-actions[bot] avatar Jun 21 '22 18:06 github-actions[bot]

📝 Docs preview for commit b07569d1708eff6fb5aacb05aab023d64c23a719 at: https://62b7196e1aa56f7e7005c960--fastapi.netlify.app

github-actions[bot] avatar Jun 25 '22 14:06 github-actions[bot]

📝 Docs preview for commit 9c3dda986c87e3e012072483e4dec85a441f8e03 at: https://62b71fa51aa56f036105c86c--fastapi.netlify.app

github-actions[bot] avatar Jun 25 '22 14:06 github-actions[bot]

📝 Docs preview for commit 232394d001e851cb52b64e0407e990ce69ba3be1 at: https://62b720b7f57baf090dfb812e--fastapi.netlify.app

github-actions[bot] avatar Jun 25 '22 14:06 github-actions[bot]

Ready to go!

Riuzaky77 avatar Jun 25 '22 14:06 Riuzaky77

📝 Docs preview for commit cb0305e1ab6f000aa5f0d7a1c9fa73629ed31c7c at: https://62b722f3e2b0760a373ddeb6--fastapi.netlify.app

github-actions[bot] avatar Jun 25 '22 15:06 github-actions[bot]

📝 Docs preview for commit f02c4aaf1f0a256607c9302ef0891ec400584946 at: https://62d18ce54624d43f5615452b--fastapi.netlify.app

github-actions[bot] avatar Jul 15 '22 15:07 github-actions[bot]

📝 Docs preview for commit a1893f053b9a4084cc134b49e0a6744755420a44 at: https://62e29b96e3bee326f645bf5b--fastapi.netlify.app

github-actions[bot] avatar Jul 28 '22 14:07 github-actions[bot]

📝 Docs preview for commit 7090cdbf9312b61f0e01445436d08f2fee2bc9b7 at: https://630e1bf89f0bdf6afee85fc5--fastapi.netlify.app

github-actions[bot] avatar Aug 30 '22 14:08 github-actions[bot]

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 avatar Oct 20 '22 19:10 kokolem

@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 :)

JarroVGIT avatar Oct 20 '22 19:10 JarroVGIT

could some docs be added to the PR?

petri avatar Nov 22 '22 22:11 petri

Would it be appropriate to add some info about using this flow on the advanced/security/oauth2-scopes/ docs page?

Ryandaydev avatar Jan 03 '23 05:01 Ryandaydev

@tiangolo is there any plans to implement client credentials flow?

nkukard avatar Feb 21 '24 04:02 nkukard