Get Local / Remote Ip address from Request?

I think this cannot be implemented in 1.0, as it is a new feature.

Maybe milestone should be changed to 1.1 or 1.2?

This is indeed a new feature.

(In [10060]) 1.2: added ->getRemoteAddress() to get the client's IP address and ->getForwardedRemoteAddress() to get the IP address from behind a proxy (closes #2798)

Note: I am not sure if ->getForwardedRemoteAddress() is the best method name. Welcome to suggestions in ticket #2798.

I'am very sorry for delay, but some comments:

Http forwarded can include not only ONE ip address, so:

HTTP_X_FORWARDED_FOR = IPaddress1, IPaddress2

is normal situation. and these IP's are not obviously remote, often they are LOCAL network ip's - so ForwardedRemoteAddress? is not really good name for this.

also one more thing, in modern web server patterns usual situation when SYMFONY is on the backend server, and on frontend is something lighty like nginx or lighttpd, just to handle connections.

in this way, HTTP ENV will have 3 IP sets

HTTP_X-REAL-IP - Real user IP, this var is defined in FRONTEND httpd server, so it can have any NAME but now it's X-REAL-IP.
HTTP_REMOTE_ADDR - IP address of FRONTEND SERVER from where request was sent to backend one
X_FORWARDED_FOR - this go unchanged or sometimes pipls useing it instead of HTTP_X-REAL-IP in frontend config, by just adding new REAL ip as first one in this list

so as you can see, REMOTE_ADDR can be "moved" to some other variable - like HTTP_X-REAL-IP, and the meaning of REMOTE_ADDR will be changed - so your getRemoteAddr() will return not user IP, but IP of frontend server. so the great IDEA to define the name of USER ip somewhere in the configs - what you think about that?

So in one of YAML file, we can set - variable name, what include CLIENT IP address, and after that getRemoteAddr - will return ip from this variable, if it's DEFINED, if nope - then from HTTP_REMOTE_ADDR. and about the names ;)

ofcourse getForwardedRemoteAddress() looks not elegant at all :-((( some my suggestions you maybe will choose:

->getRemoteAddr();
->getForwardedAddr();

->getRemoteAddr();
->getXForwardedFor();

->getRemoteIp();
->getLocalIp();

->getIp(); // by default will return remote
->getIp("X_FORWARDED");
->getIp("X_FORWARDED_REAL");

or in object style:
->getIp(sfIP::X-REAL-IP);
->getIp(sfIP::FORWARDED);

This patch should do the trick. ->getForwardedRemoteAddress() is now ->getForwardedFor(), and returns an array or null.

(In [13018]) [1.2] fixed #2798 by applying Patch from Nebelmann