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

[REFACTOR][ENHANCEMENT] argilla server: better OAuth2 integration #5689

Open
wants to merge 31 commits into
base: develop
Choose a base branch
from

fix: Using backend.settings

125889b
Select commit
Loading
Failed to load commit list.
Open

[REFACTOR][ENHANCEMENT] argilla server: better OAuth2 integration #5689

fix: Using backend.settings
125889b
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Nov 14, 2024 in 0s

92.39% of diff hit (target 91.08%)

View this Pull Request on Codecov

92.39% of diff hit (target 91.08%)

Annotations

Check warning on line 22 in argilla-server/src/argilla_server/security/authentication/oauth2/auth_backend.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

argilla-server/src/argilla_server/security/authentication/oauth2/auth_backend.py#L22 

Added line #L22 was not covered by tests

Check warning on line 42 in argilla-server/src/argilla_server/security/authentication/oauth2/auth_backend.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

argilla-server/src/argilla_server/security/authentication/oauth2/auth_backend.py#L42 

Added line #L42 was not covered by tests

Check warning on line 131 in argilla-server/src/argilla_server/security/authentication/oauth2/provider.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

argilla-server/src/argilla_server/security/authentication/oauth2/provider.py#L131 

Added line #L131 was not covered by tests

Check warning on line 134 in argilla-server/src/argilla_server/security/authentication/oauth2/provider.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

argilla-server/src/argilla_server/security/authentication/oauth2/provider.py#L133-L134 

Added lines #L133 - L134 were not covered by tests

Check warning on line 136 in argilla-server/src/argilla_server/security/authentication/oauth2/provider.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

argilla-server/src/argilla_server/security/authentication/oauth2/provider.py#L136 

Added line #L136 was not covered by tests

Check warning on line 41 in argilla-server/src/argilla_server/validators/users.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

argilla-server/src/argilla_server/validators/users.py#L41 

Added line #L41 was not covered by tests
View more details on Codecov