Tim Taubert [:ttaubert] (inactive) Assignee
	Description • 10 years ago

SHA-224 support has been removed from the spec:

https://dvcs.w3.org/hg/webcrypto-api/rev/3f7df730b2c7

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 1 • 10 years ago

I based this off of bug 1020882.

	Richard Barnes [:rbarnes]
	Comment 2 • 10 years ago

Comment on attachment 8435657 [details] [diff] [review]
0002-Bug-1021601-Remove-SHA-224-support-from-WebCrypto-AP.patch

Review of attachment 8435657 [details] [diff] [review]:
-----------------------------------------------------------------

Just to be clear: I'm OK removing this hash function, but it is not true in general that we have to be limited to the algorithms listed in the spec.  If we had some reason to support, say, RIPEMD-160, we should not feel constrained by the fact that it's not in the spec.

	Dana Keeler (she/her) (use needinfo) (:keeler for reviews)
	Comment 3 • 10 years ago

Comment on attachment 8435657 [details] [diff] [review]
0002-Bug-1021601-Remove-SHA-224-support-from-WebCrypto-AP.patch

Review of attachment 8435657 [details] [diff] [review]:
-----------------------------------------------------------------

I couldn't seem to find tests for if webcrypto is given a hash algorithm it doesn't support - we should add some.

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 4 • 10 years ago

(In reply to Richard Barnes [:rbarnes] from comment #2)
> Just to be clear: I'm OK removing this hash function, but it is not true in
> general that we have to be limited to the algorithms listed in the spec.  If
> we had some reason to support, say, RIPEMD-160, we should not feel
> constrained by the fact that it's not in the spec.

Right, I'm not totally familiar with the spec yet, thanks for clarifying!

(In reply to David Keeler (:keeler) [use needinfo?] from comment #3)
> I couldn't seem to find tests for if webcrypto is given a hash algorithm it
> doesn't support - we should add some.

Yes, we should.

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 5 • 10 years ago

(In reply to David Keeler (:keeler) [use needinfo?] from comment #3)
> I couldn't seem to find tests for if webcrypto is given a hash algorithm it
> doesn't support - we should add some.

Actually, bug 1020882 that I just landed contains a test for that.

	Tim Taubert [:ttaubert] (inactive) Assignee
	Comment 6 • 10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/3efccd28317b

	Ryan VanderMeulen [:RyanVM]
	Comment 7 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/3efccd28317b