XChange
XChange copied to clipboard
Support for tinyproxy at the connect method of NettyStreamingExchange
Overview
Due the firm I work form's use of reverse proxy extensively based on tinyproxy. deploying and connecting to a number of providers such as kraken,binance, coinbasepro we had to basically modify several classes to support wss connection stage changes were made NettyStreamingService
- for handling proxy settings and connect initiattion BitstampStreamingExchange to ensure the default implementation of applyStreamingSpecification in StreamingExchange is invoked
below is a snippet of StreamingExchange, as modified default void applyStreamingSpecification(ExchangeSpecification exchangeSpec, NettyStreamingService<?> streamingService){ streamingService.setSocksProxyHost((String) exchangeSpec.getExchangeSpecificParametersItem(SOCKS_PROXY_HOST)); streamingService.setSocksProxyPort((Integer) exchangeSpec.getExchangeSpecificParametersItem(SOCKS_PROXY_PORT)); streamingService.setHttpProxyHost(exchangeSpec.getProxyHost()); streamingService.setHttpProxyPort(exchangeSpec.getProxyPort()); streamingService.setBeforeConnectionHandler((Runnable) exchangeSpec.getExchangeSpecificParametersItem(ConnectableService.BEFORE_CONNECTION_HANDLER));
Boolean accept_all_ceriticates = (Boolean) exchangeSpec.getExchangeSpecificParametersItem(ACCEPT_ALL_CERITICATES);
Hi @scrisini - this sounds good. Would you be willing to raise a PR with the changes? I'm sure others would find them useful.
Hi @badgerwithagun great! will do. im a newbie on this so I just clone the latest and follow the guidelines and then ready I submit for peer review and PR?
Yes, @scrisini, don't worry about making mistakes. You can't break anything ;).
Someone will let you know if your PR needs work.
Hi has taken me a while to catchup. but im ready to create a PR but not sure how this is done here
- I cloned the repo today
- branched off from develop - named it tinyproxy_3647
- however im now ready to push to GitHub and do a PR
- but looks like my username isn't permitted to do that? Cheers

Yes, @scrisini, don't worry about making mistakes. You can't break anything ;).
Someone will let you know if your PR needs work.
Hello just following this up as I now have incorporated the fix to the latest cloned copy and would like to know whats next step to create a PR? thank you.
google it. lot's of info out there already on creating a PR on github
Thanks! I should have read a bit more ! all sorted - I missed the fork step.