BC: Add server params (REMOTE_ADDR) to PSR request

[bohanwood]

Because we need the client IP.
Even if X-Forwarded-For or X-Real-IP header can be used,
it's important to use the REMOTE_ADDR to determine whether
the request comes from a trusted edge server/proxy.

[dominikzogg]

Hello @bohanyang

Thank you for your PR.

Have you checked if there is no way getting this information from the workerman request itself?
I ask, cause this is a BC, means merging an release this change will mean to release a new major version.

Regards Dominik

[bohanwood]

@dominikzogg

Yeah, it has to cause a BC break since the WorkermanRequest contains no information beyond HTTP header lines and body.

I've updated the parameter order according to your suggestion.

[dominikzogg]

Please update https://github.com/chubbyphp/chubbyphp-workerman-request-handler/blob/master/composer.json#L53 and the version in the readme

[bohanwood]

Version updated

[dominikzogg]

@bohanyang sorry, i should have been more precise, version needs to be 2.0

[bohanwood]

Oh sorry it's a major version update, my fault.
Updated

[dominikzogg]

@bohanyang it seems that the CI doesn't get started on the CI, have you run the tests locally? I guess the OnMessageTest should fail.

[bohanwood]

Tests will pass now.

[dominikzogg]

@bohanyang thanks for your contribution

[dominikzogg]

https://github.com/chubbyphp/chubbyphp-workerman-request-handler/releases/tag/2.0.0