Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FileHandler::getRemoteResource() 에서 프록시 서버를 제대로 사용하지 못하는 문제 수정 #1921

Merged
3 commits merged into from
Jul 15, 2016

Conversation

kijin
Copy link
Contributor

@kijin kijin commented Jun 18, 2016

XE에서는 __PROXY_SERVER__ 상수를 사용해서 외부 요청시 경유할 프록시 서버를 지정할 수 있으나, 파일핸들러에서 일반적인 프록시 사용 방식이 아닌 이상한 방식으로 처리하고 있어 사실상 무용지물입니다. 프록시 기능을 제대로 지원하지 않던 구버전 PEAR HTTP_Request 클래스에 맞추어 개발한 후, HTTP_Request 클래스를 업데이트하면서 잊어버린 것 같아요.

특정 URL에 POST하는 기존 방식의 프록시 설정은 호환성 유지를 위해 그대로 두고, 123.45.67.89:8080 포맷으로 프록시를 지정하면 HTTP_Request 클래스에서 공식 지원하는 설정을 통해 정상 처리하도록 변경해 보았습니다.

@bjrambo
Copy link
Contributor

bjrambo commented Jun 18, 2016

저번에 제가 알려드린 항목도 여기에 같이 추가해두면 좋을듯?

@kijin
Copy link
Contributor Author

kijin commented Jun 18, 2016

@bjrambo 추가했습니다.

@ghost ghost self-assigned this Jul 11, 2016
@ghost ghost modified the milestone: 1.8.23 Jul 14, 2016
@ghost ghost merged commit b88639b into xpressengine:develop Jul 15, 2016
@kijin kijin deleted the fix/proxy-server branch August 12, 2016 05:30
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants