koajs/koa

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

Open

#827 geöffnet am 4. Okt. 2016

Auf GitHub ansehen
 (12 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (35.713 Stars) (3.215 Forks)batch import
help wanted

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

Tech Stack
javascriptnodejs
Domain
backend
Issue Type
bug
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.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
basic JavaScriptunderstanding of HTTP headers
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80
Research-Richtung
Focus on Koa's request object, specifically the 'ip' getter in lib/request.js. The x forwarded for header may include a port (e.g., '23.243.1.1:38242'). The task is to strip the port from the IP address. Look at the existing code for parsing the header and ensure only the IP is returned.