added http proxy support #5
Conversation
| @@ -1,3 +1,3 @@ | |||
| class TelegramBotMiddleware | |||
| VERSION = "0.3.2" | |||
| VERSION = "0.3.3" | |||
There was a problem hiding this comment.
Style/MutableConstant: Freeze mutable objects assigned to constants.
| log_debug("Sending to chat: #{path} - #{query}") | ||
| self.class.post("/bot#{@config.token}/#{path}", query: query) | ||
| options=@def_options.clone | ||
| options[:query]=query |
There was a problem hiding this comment.
Layout/SpaceAroundOperators: Surrounding space missing for operator =.
| def send_to_telegram(path, query) | ||
| log_debug("Sending to chat: #{path} - #{query}") | ||
| self.class.post("/bot#{@config.token}/#{path}", query: query) | ||
| options=@def_options.clone |
There was a problem hiding this comment.
Layout/SpaceAroundOperators: Surrounding space missing for operator =.
| @def_options[:http_proxyport]=(@config.proxy_port || 1080).to_i | ||
| if ! @config.proxy_user.nil? | ||
| @def_options[:http_proxyuser]=@config.proxy_user | ||
| @def_options[:http_proxypass]=@config.proxy_pass || '' |
There was a problem hiding this comment.
Layout/SpaceAroundOperators: Surrounding space missing for operator =.
Style/StringLiterals: Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
| @def_options[:http_proxyaddr]=@config.proxy_addr | ||
| @def_options[:http_proxyport]=(@config.proxy_port || 1080).to_i | ||
| if ! @config.proxy_user.nil? | ||
| @def_options[:http_proxyuser]=@config.proxy_user |
There was a problem hiding this comment.
Layout/SpaceAroundOperators: Surrounding space missing for operator =.
| if ! @config.proxy_addr.nil? | ||
| @def_options[:http_proxyaddr]=@config.proxy_addr | ||
| @def_options[:http_proxyport]=(@config.proxy_port || 1080).to_i | ||
| if ! @config.proxy_user.nil? |
There was a problem hiding this comment.
Style/NegatedIf: Favor unless over if for negative conditions.
Layout/SpaceAfterNot: Do not leave space between ! and its argument.
| @def_options={} | ||
| if ! @config.proxy_addr.nil? | ||
| @def_options[:http_proxyaddr]=@config.proxy_addr | ||
| @def_options[:http_proxyport]=(@config.proxy_port || 1080).to_i |
There was a problem hiding this comment.
Layout/SpaceAroundOperators: Surrounding space missing for operator =.
| # proxy settings | ||
| @def_options={} | ||
| if ! @config.proxy_addr.nil? | ||
| @def_options[:http_proxyaddr]=@config.proxy_addr |
There was a problem hiding this comment.
Layout/SpaceAroundOperators: Surrounding space missing for operator =.
|
|
||
| # proxy settings | ||
| @def_options={} | ||
| if ! @config.proxy_addr.nil? |
There was a problem hiding this comment.
Style/NegatedIf: Favor unless over if for negative conditions.
Layout/SpaceAfterNot: Do not leave space between ! and its argument.
| raise ArgumentError.new("Config error: token can't be null || empty.") if @config.token.nil? || @config.token.empty? | ||
|
|
||
| # proxy settings | ||
| @def_options={} |
There was a problem hiding this comment.
Layout/SpaceAroundOperators: Surrounding space missing for operator =.
2 similar comments
3 participants
simple and fast (but working well) support for http proxy.