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

Accept raw string confirmation code on confirm device #240

Merged

Conversation

MarcusGrass
Copy link
Contributor

Got a panic here today, seems to have been changed recently.

Removes u32 validation, since it seems signal isn't sticking to u32 codes. Besides, it's just converting to a u32 to then converting back to a string on the url. Backwards compatible.

@codecov
Copy link

codecov bot commented Aug 5, 2023

Codecov Report

Merging #240 (101495a) into main (6d2649d) will not change coverage.
Report is 2 commits behind head on main.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main    #240   +/-   ##
=====================================
  Coverage   4.16%   4.16%           
=====================================
  Files         36      36           
  Lines       2523    2523           
=====================================
  Hits         105     105           
  Misses      2418    2418           
Files Changed Coverage Δ
libsignal-service/src/provisioning/manager.rs 0.00% <ø> (ø)

@rubdos
Copy link
Member

rubdos commented Aug 7, 2023

Cherry-picked into #238

@rubdos rubdos closed this Aug 7, 2023
@rubdos rubdos reopened this Aug 7, 2023
@rubdos rubdos enabled auto-merge August 7, 2023 11:59
@rubdos rubdos disabled auto-merge August 7, 2023 11:59
@rubdos rubdos enabled auto-merge (squash) August 7, 2023 12:00
@rubdos rubdos merged commit 81eb911 into whisperfish:main Aug 7, 2023
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants