Base64Url Id instead of byte[]

[Regenhardt]

#513

Not sure if any of the places that now use RawId should be changed to use Id instead, like CredentialId being returned from a successful assertion. This basically just changes the model and makes everything use the RawId, so minimal breaking changes hopefully.

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.03%. Comparing base (b7f1735) to head (61e2694).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #586   +/-   ##
=======================================
  Coverage   76.03%   76.03%           
=======================================
  Files         101      101           
  Lines        2600     2600           
  Branches      432      432           
=======================================
  Hits         1977     1977           
  Misses        513      513           
  Partials      110      110           

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

[abergs]

I've been reviewing this, as well has reading some of the related WebAuthn discussions. like: w3c/webauthn#2119

Although this will introduce lots of "won't-compile"-errors when people upgrade to 4.0, I agree it's probably the right thing to do.

The current set of changes look good, however I think we also need to change the Id-prop on AuthenticatorAttestationRawResponse, so I will request changes.