Custom for fuster #37

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

Open

VITca64rus wants to merge 1 commit into marcelo225:master from VITca64rus:customize_for_fuster

VITca64rus commented Feb 9, 2024

No description provided.


          Custom for fuster

05beef3

chmoder mentioned this pull request

aud is using client_id by default #32

Closed

Owner

marcelo225 commented Feb 26, 2024 •

edited

Loading

@VITca64rus , have you ever reviewed the tests about your modifications?

As you have changed considerably, after running the tests, I've realized it has some errors in some tests:

test_when_no_role_is_present_none_is_returned
test_when_only_client_roles_are_present_client_roles_are_returned
test_when_only_realm_roles_are_present_realm_roles_are_returned
test_when_realm_roles_and_client_roles_are_present_both_are_returned
test_when_token_as_active_and_has_roles_request_authorizated
test_when_token_as_active_and_has_roles_request_not_authorizated
test_when_token_is_active_and_has_roles_request_authorizated

Could you check these out?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet