Alex Seigler
Alex Seigler
All thrown exceptions currently have messages now, but have different formats.
This looks really familiar...I thought I had run into and fixed this on branch [demo-update](https://github.com/passwordless-lib/fido2-net-lib/tree/demo-update), but I don't see it currently. We would certainly accept a PR here.
Fixed by #290. Thank you!
Did you see the [demo site](https://github.com/abergs/fido2-net-lib/tree/master/Demo)? It's the code behind https://passwordless.dev. Or maybe I am misunderstanding the question.
Perhaps we bump this to 1.18 now that 1.17 is no longer supported and try to get it merged?
I think this is ready to merge @MasterKale unless anyone has objections? This is what is breaking actions currently.
The metadata tests probably need to be completely overhauled, lots of changes occurred in them since the tests were initially implemented, I am not surprised they are all broken now.
Ah, yes. There is a link in that post to a [bit of javascript](https://github.com/webauthn-open-source/fido2-lib/blob/f9a0f8bbb3b173f17b1664df7263d7bda8bce424/lib/attestations/tpm.js#L173) that tells the story. ``` // TPMU_PUBLIC_PARMS parameters if (type !== "TPM_ALG_RSA") { throw new Error("tpm...
Looks good! You can lose the 14, 16, and 18 cases in `getUnique(),` those don't apply to this TPM usage. You'll only see TPM_ALG_RSA or TPM_ALG_ECC there.
That's not a bad idea. I can't see why we could ever need to do that import. In fact, I am struggling to recall why I grabbed just the tiny...