php / php/php-src

Unify case of array keys in `ftp_mlsd`

Aperta
#18,064 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Extension: ftp Feature Status: Needs Triage
Lingua principale
C
Stelle
40.4k
Fork
8.1k
Merge medio
2g 13h
PR unite (30g)
96

Descrizione

Description

Function ftp_mlsd started returning array with different key case: "name" is still "name", but "type" became "Type" and "modify" became "Modify".

I assume ftp_mlsd returns data as provided by the FTP server and this change was a consequence of our hosting provider modifying the software on FTP server, and not a consequence of changes in PHP.

RFC 3659 states in section 7.5 that fact names are case-insensitive, but because PHP array keys are not, changes like this break the code.

I suggest casing of array keys is unified (always "type" or always "Type").

Guida per i contributori

Apri la guida per i contributori

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.

Direzione di ricerca

Inizia tracciando l’implementazione di ftp_mlsd e il modo in cui i nomi dei fact FTP vengono convertiti in chiavi di array PHP. Leggi la sezione 7.5 di RFC 3659 insieme al comportamento esistente, quindi verifica che le chiavi restituite usino una capitalizzazione coerente indipendentemente dalla risposta del server.

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

Valutazione

Stack tecnologico
c, php
Ambito
networking
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.