koajs/koa

request.ip via x-forwarded-for should strip out ports

Offen

#827 geöffnet am 04.10.2016

Auf GitHub ansehenContributor Guide
 (12 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (3.215 Forks)batch import
help wanted

Repository-Metriken

Stars
 (35.713 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 9h 19m) (2 gemergte PRs in 30 T)

Beschreibung

on azure (yes, i know), ports are also being sent as well as the ip in x-forwarded-for:

'x-forwarded-for': '23.243.1.1:38242'

request.ip should not return the port section. aye or nay?

Contributor Guide

Research-Richtung
Finde heraus, wo koa den x forwarded for Header parst und entferne den Port, falls vorhanden.
Tech Stack
javascriptnodejs
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
HTTP headersGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80