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

open-uriでno_proxy環境変数も使用できるので追加 #2857

Merged
merged 1 commit into from
May 2, 2024

Conversation

sada
Copy link
Contributor

@sada sada commented Dec 16, 2023

概要

#484 のissueに関するPRです。

open-urino_proxy 環境変数をデフォルトで使用できるので、プロキシを使用する場合の環境変数の説明に追加しました。

Ruby 2.7.0のURIの find_proxy でも no_proxy 環境変数が使用できます。
open-uri は内部で上記を呼び出していますので、現在のRubyリファレンスマニュアルのサポート対象がRuby 2.7~3.2のようですのでドキュメントの説明にそのまま追加しました。

https://github.com/ruby/ruby/blob/v2_7_0/lib/uri/generic.rb#L1475-L1489

Copy link
Member

@znz znz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます。
遅くなりましたが、マージします。

@znz znz merged commit 45fb296 into rurema:master May 2, 2024
8 checks passed
@sada sada deleted the add_no_proxy_for_open_uri branch May 3, 2024 02:12
@sada
Copy link
Contributor Author

sada commented May 3, 2024

マージありがとうございます!

お手数をお掛けしますが、対応済みとして #484 のissueを閉じていただけないでしょうか。
私は権限がないので閉じれません。

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