-
Notifications
You must be signed in to change notification settings - Fork 327
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.
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
4024/passkeys #4134
base: master
Are you sure you want to change the base?
4024/passkeys #4134
Conversation
Hello @nilsbehlen! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2025-01-15 11:37:38 UTC |
# Conflicts: # privacyidea/api/lib/prepolicy.py # privacyidea/api/token.py # tests/test_lib_events.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit
codespell
tests/test_api_passkey.py|217| assertIn ==> asserting, assert in, assertion
tests/test_api_passkey.py|228| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|82| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|83| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|84| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|114| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|115| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|116| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|118| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|121| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|123| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|125| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|126| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|128| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|147| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|148| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|155| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|203| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|207| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|235| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|242| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|255| assertIn ==> asserting, assert in, assertion
tests/test_lib_tokens_passkeytoken.py|270| assertIn ==> asserting, assert in, assertion
# Conflicts: # requirements.txt # tests/requirements.txt
# Conflicts: # privacyidea/api/token.py # tests/test_api_lib_policy.py
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #4134 /- ##
==========================================
- Coverage 92.40% 92.40% -0.01%
==========================================
Files 205 210 5
Lines 27447 27854 407
==========================================
Hits 25363 25738 375
- Misses 2084 2116 32 ☔ View full report in Codecov by Sentry. |
# Conflicts: # privacyidea/lib/token.py # privacyidea/lib/tokens/pushtoken.py # tests/test_lib_events.py
the implementation is provisionally
if no machine name is found in the user agent, the offline data will not be in the response and a refill is not possible either
No description provided.