rotty3000/liferay-portal — pull request #254 by victormirandabeltran (2013-07-03 11:38:08+00:00)
Title: LPS-35464 Send the proper port to checkConnect method

[Changed LPS code in the git commit]

Hey Ray.

This is a small fix in the PortalSocketPermission.checkConnect method, that is used when PACL is enabled and, for example, we want to send a callback for a blog entry.

The root problem is that the checkConnect method selects always the port 80/443, even if the url has a port like 8080.

However, in master there is another error with the trackbacks that is not reproduced in 6.1.x. It is important to notice that this second error doesn't have relation with this fix and its cause.

We need to backport this as soon as possible because this error is a release blocker of GA3 release (the release has to be completed this week), so the last part of the fix that is only related with master will be included in another commit.

PD: There is another problem related with HttpImpl.getDomain(String url). This method returns the port too. I will fix this problem in master in a separate commit (this commit will be backported as I told you before, and we couldn't change the behaviour of a public api in a backport)

If you have doubt, please do not hesitate to ask me.

Thanks.