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.

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

feat: Support more types of mnemonics #855

Open
wants to merge 2 commits into
base: artonge/feat/validate_user_public_keys
Choose a base branch
from

Conversation

artonge
Copy link
Collaborator

@artonge artonge commented Jan 8, 2025

Other clients moved to a more secure hash and iteration count, so we need to accept those.

@artonge artonge added 3. to review Waiting for reviews enhancement New feature or request javascript Javascript related ticket labels Jan 8, 2025
@artonge artonge added this to the Nextcloud 31 milestone Jan 8, 2025
@artonge artonge requested review from nfebe and skjnldsv January 8, 2025 16:53
@artonge artonge self-assigned this Jan 8, 2025
@artonge artonge force-pushed the artonge/feat/validate_user_public_keys branch from 25c43be to 3704510 Compare January 8, 2025 16:55
@artonge artonge force-pushed the artonge/feat/support_more_mneomonic branch from 3aa9cf0 to 3fc7f13 Compare January 8, 2025 16:56
@artonge artonge force-pushed the artonge/feat/validate_user_public_keys branch from 3704510 to 51f8f44 Compare January 9, 2025 09:40
@artonge artonge force-pushed the artonge/feat/support_more_mneomonic branch from 3fc7f13 to 37551bf Compare January 9, 2025 09:41
@artonge artonge force-pushed the artonge/feat/validate_user_public_keys branch 2 times, most recently from 5584e77 to 1bd3dd4 Compare January 9, 2025 17:06
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
@artonge artonge force-pushed the artonge/feat/support_more_mneomonic branch from 37551bf to b6a7c27 Compare January 9, 2025 17:14
Copy link

codecov bot commented Jan 9, 2025

Codecov Report

Attention: Patch coverage is 94.87179% with 2 lines in your changes missing coverage. Please review.

Project coverage is 26.18%. Comparing base (1bd3dd4) to head (b6a7c27).

Files with missing lines Patch % Lines
src/services/privateKeyUtils.ts 90.00% 2 Missing ⚠️
Additional details and impacted files
@@                            Coverage Diff                             @@
##           artonge/feat/validate_user_public_keys     #855      +/-   ##
==========================================================================
+ Coverage                                   25.73%   26.18%   +0.44%     
==========================================================================
  Files                                          30       30              
  Lines                                        1321     1333      +12     
  Branches                                       32       37       +5     
==========================================================================
+ Hits                                          340      349       +9     
- Misses                                        978      981       +3     
  Partials                                        3        3              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews enhancement New feature or request javascript Javascript related ticket
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants