pypi/warehouse

Update all IP Addresses in tests to use RFC-5735 TEST-NET-* addresses.

Aperta

#13.186 aperta il 10 mar 2023

Vedi su GitHubGuida contributor
 (3 commenti) (1 reazione) (0 assegnatari)Python (1201 fork)auto 404
good first issuetesting

Metriche repository

Star
 (4097 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

nitpick: ‏super nitpicky, and not the first violation of this type in the codebase, but per RFC-5735 we ought to use addresses in TEST-NET-* ranges. Thankfully we have pytest-socket (#humbebrag) to prevent actual network calls, so it's not an issue, but I wanted to note it while it was fresh in my head. (Also see https://www.rfc-editor.org/rfc/rfc6890#section-2.2.2) Might be worth a full sweep cleanup as a separate PR.

Originally posted by @miketheman in https://github.com/pypi/warehouse/pull/13164#discussion_r1131763629

Guida contributor

Direzione di ricerca
Identifica tutti gli indirizzi IP utilizzati nei test che non sono nei range RFC 5735 TEST NET, sostituiscili con indirizzi di test validi (ad esempio 192.0.2.0/24, 198.51.100.0/24, 203.0.113.0/24) e verifica che i test siano superati.
Tech stack
python
Dominio
testing
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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
PythonGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75