quic: session.peerCertificate behaves differently than for http2
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 122k
- Forks
- 37.3k
- Ø Merge
- 4 T. 2 Std.
- Gemergte PRs (30 T.)
- 283
Beschreibung
I am currently implementing a custom certificate verifier.
So I wait for the sesssion.opened promise to be fullfilled, and try to check the peerCertificate.
It is an X509Certificate {} object.
I wanted to check the fingerprint256 property. For quic it is a function, for http2 it was a property (supposedly a getter).
The current doc says:
The peer's certificate as an object with properties such as subject, issuer, valid_from, valid_to, fingerprint, etc. Returns undefined if the session is destroyed or the peer did not present a certificate.
So I think the intention is that it is also a property for quic, but it is not.
Also the docu for X509Certificate says:
x509.fingerprint256#
Added in: v15.6.0
Type:
The SHA-256 fingerprint of this certificate.
Though I can see easily what differs between these implementations.
(Of course not a big deal).
@jasnell is this really an issue?
Probably this applies to other properties as well.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginnen Sie mit den QUIC- und HTTP/2-Einstiegspunkten session.peerCertificate und vergleichen Sie, wie X509Certificate.fingerprint256 bereitgestellt wird. Prüfen Sie die zugehörige Dokumentation zu X509Certificate und weitere Peer-Zertifikatseigenschaften. Verifizieren Sie anschließend, dass der beabsichtigte Zugriffsmodus über die Protokolle hinweg konsistent ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- networking
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Aktiv
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 55/100