devsisters / goquic

QUIC support for Go
http://devsisters.github.io/goquic/
BSD 3-Clause "New" or "Revised" License
942 stars 100 forks source link

fix example reverse proxy server #50

Closed cnabro closed 7 years ago

cnabro commented 7 years ago

reverse proxy테스트 중, 기존 예제 proxy server의 예제에서 flag 옵션이 정상적으로 반영되지 않아, proxy_url flag로 동작하도록 수정하였습니다.

as is ) sudo ./reverse_proxy "http://test.com" -cert server.crt -key server.key -port 433

to be) sudo ./reverse_proxy -proxy_url "http://test.com" -cert server.crt -key server.key -port 433

bzz commented 7 years ago

\cc @hodduc for review

serialx commented 7 years ago

패치 보내주셔서 감사합니다. 한 가지 확인 좀 부탁드립니다.

URL을 뒤에 넣어도 작동이 안되시나요? 보통 명령어 [옵션] 인자 순으로 입력받습니다. 예시:

sudo ./reverse_proxy -cert server.crt -key server.key -port 433 "http://test.com"

한 번 테스트 부탁드립니다.

cnabro commented 7 years ago

@serialx 방금 말씀하신대로 해보니 잘되는듯 합니다. 인자를 먼저 보내서 그런것 같네요, 감사합니다

sudo ./reverse_proxy -cert server.crt -key server_nopass.key -port 443 "http://test.com"
2017/02/08 15:54:39 About to listen on 0.0.0.0. Go to https://0.0.0.0:443/ 2017/02/08 15:54:39 Starting reverse proxy for backend URL: http://test.com [0208/155439:VERBOSE1:quic_crypto_server_config.cc(982)] New primary config. orbit: E321EEA5DC1EC72D scid: 3E49447BCD612BE4266C5F5BA5188389