Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Oauth2 provider #133

Merged
merged 46 commits into from
May 8, 2024
Merged

Oauth2 provider #133

merged 46 commits into from
May 8, 2024

Conversation

john-westcott-iv
Copy link
Member

@john-westcott-iv john-westcott-iv commented Feb 11, 2024 •
edited
Loading

Replaces #44

  • Update requirements_all.txt
  • Add common model changes to prevent time changing from oauth2 system changes
  • Add tests
  • Fix linting

@john-westcott-iv john-westcott-iv force-pushed the oauth2_provider_new branch 4 times, most recently from e4d7512 to 2586a96 Compare February 12, 2024 12:50
@john-westcott-iv john-westcott-iv force-pushed the oauth2_provider_new branch 2 times, most recently from 22a6fde to 5175a48 Compare February 28, 2024 19:41
@john-westcott-iv john-westcott-iv force-pushed the oauth2_provider_new branch 3 times, most recently from 4231acd to b56bc93 Compare April 15, 2024 19:29
@relrod relrod self-assigned this Apr 16, 2024
@relrod relrod force-pushed the oauth2_provider_new branch 5 times, most recently from e3a5793 to 5dbf5b2 Compare April 22, 2024 22:07
@relrod relrod force-pushed the oauth2_provider_new branch 4 times, most recently from c596543 to 7d35c33 Compare May 1, 2024 20:31
john-westcott-iv and others added 9 commits May 5, 2024 17:09
relrod added 8 commits May 5, 2024 17:09
@relrod
Use CommonModel fields for created and modified
33754a2 
We have to use the right order of inheritance for this to work, and
not null out fields that we actually want :)

Signed-off-by: Rick Elrod <[email protected]>
john-westcott-iv
john-westcott-iv commented May 7, 2024
View reviewed changes
ansible_base/oauth2_provider/models/application.py Show resolved Hide resolved
ansible_base/lib/serializers/common.py Show resolved Hide resolved
ansible_base/lib/utils/views/ansible_base.py Show resolved Hide resolved
ansible_base/oauth2_provider/models/access_token.py Show resolved Hide resolved
ansible_base/oauth2_provider/models/access_token.py Show resolved Hide resolved
ansible_base/oauth2_provider/serializers/application.py Show resolved Hide resolved
ansible_base/oauth2_provider/serializers/token.py Outdated Show resolved Hide resolved
ansible_base/oauth2_provider/serializers/token.py Outdated Show resolved Hide resolved
ansible_base/oauth2_provider/serializers/token.py Outdated Show resolved Hide resolved
ansible_base/oauth2_provider/views/token.py Outdated Show resolved Hide resolved
This was referenced May 8, 2024
Open
Closed
relrod added 2 commits May 8, 2024 16:10
@relrod
Nix dependency LB
3e685c3 
Signed-off-by: Rick Elrod <[email protected]>
@relrod
Use wildcard
f43a778 
Signed-off-by: Rick Elrod <[email protected]>
@relrod relrod mentioned this pull request May 8, 2024
Open
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented May 8, 2024

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
90.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@john-westcott-iv john-westcott-iv changed the title [WIP] Oauth2 provider Oauth2 provider May 8, 2024
@john-westcott-iv john-westcott-iv mentioned this pull request May 8, 2024
Open
@relrod relrod merged commit 010efea into ansible:devel May 8, 2024
8 checks passed
thedoubl3j pushed a commit to thedoubl3j/django-ansible-base that referenced this pull request Jun 18, 2024
@john-westcott-iv @relrod @thedoubl3j
Oauth2 provider (ansible#133)
a855798 
Replaces ansible#44
- [x] Update requirements_all.txt
- [x] Add common model changes to prevent time changing from oauth2
system changes
- [x] Add tests
- [x] Fix linting

---------

Signed-off-by: Rick Elrod <[email protected]>
Co-authored-by: Rick Elrod <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@relrod relrod relrod left review comments

@jctanner jctanner jctanner approved these changes

Assignees

@relrod relrod

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@john-westcott-iv @relrod @jctanner