haskell/aeson

Add saner ByteString instances

Aperta

#187 aperta il 5 mar 2014

Vedi su GitHubGuida contributor
 (22 commenti) (3 reazioni) (0 assegnatari)Haskell (334 fork)batch import
blocked by upstream/should be done externallyhelp wantedneeds decision

Metriche repository

Star
 (1298 stelle)
Metriche merge PR
 (Merge medio 935g 9h) (5 PR mergiate in 30 g)

Descrizione

ByteString instances were removed because they allowed people to try and insert arbitrary binary data that JSON can't handle.

Could we not re-add ByteString instances that leveraged something like base64 encoding (which JSON can take I believe) to work around this limitation?

<mgsloan> Fuuzetsu: Best thing to do is to avoid putting bytestrings in json,
          but when it's unavoidable:
          https://gist.github.com/mgsloan/bfe4bf2250c1d656a2b1  [05:03]

Guida contributor

Direzione di ricerca
Studia il commit che ha rimosso le istanze di ByteString, comprendi l'approccio di codifica base64 nel gist collegato e implementa nuove istanze per ByteString che codificano tramite base64.
Tech stack
Nessuno
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
HaskellGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70