IPAddress toString should use existing printTo algorithm and conform to RFC 5952 canonical format for IPv6 addresses

Aperta
#228 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
55/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Ferma
Stack tecnologico
cpp
Ambito
api, networking

Direzione di ricerca

Inizia dalle implementazioni di IPAddress toString() e printTo(), quindi esamina i test printTo() esistenti per i casi IPv6 canonici di RFC 5952. Fai in modo che toString() utilizzi l’algoritmo printTo() stabilito e aggiungi unit test che ne coprano l’output; il lavoro è completo quando entrambi i metodi producono il formato canonico previsto.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

enhancement
API component

IPAddress

Description

The toString() for an IPv6 type address has a different output from printTo(), and does not follow the RFC 5952 canonical format. It has an unnecessarily long fixed length format. Note that IPv4 does not use a fixed length format.

Also note that the existing printTo() method already implements the RFC 5952 canonical format algorithm, and has extensive tests (there are no tests for the toString() method).

The recommendation would be for toString() simply to use the existing algorithm from printTo() (and add some unit tests for toString).

Is this a breaking change?

No. Both formats are valid string formats for IPv6, and fromString() handles both of them (and all other valid formats), as should any other system. The canonical format is usually shorter, but will not exceed the fixed length format, so there is no issue with overruns.

Lingua principale
C++
Stelle
306
Fork
150
Nessuna PR unita negli ultimi 30g

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di arduino/ArduinoCore-API

Tutte le issue di arduino/ArduinoCore-API

Issue simili

Altre issue su C++

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.