koajs/koa

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

Open

#827 创建于 2016年10月4日

在 GitHub 查看
 (12 评论) (0 反应) (0 负责人)JavaScript (35,713 star) (3,215 fork)batch import
help wanted

描述

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?

贡献者指南

技术栈
javascriptnodejs
领域
backend
议题类型
bug
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
under 1 hour
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
stale
清晰度议题是否清楚说明期望改动、验收标准和下一步。
clear
前置要求
basic JavaScriptunderstanding of HTTP headers
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
80
研究方向
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.