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

wget 1.19.1 #9873

Closed
wants to merge 1 commit into from
Closed

wget 1.19.1 #9873

wants to merge 1 commit into from

Conversation

zmwangx
Copy link
Contributor

@zmwangx zmwangx commented Feb 11, 2017

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

  • Make openssl mandatory and libidn recommended per discussions in wget 1.19 #9613;
  • Test: write retrieved page to /dev/null (the hundreds of kilobytes of output on stdout is useless).

Formula/wget.rb Outdated
depends_on "openssl" => :recommended
depends_on "libidn" if build.with? "iri"
depends_on "openssl"
depends_on "libidn" => :recommended
Copy link
Member

Choose a reason for hiding this comment

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

What does libidn do? Not sure what IRI support is but I'd personally rather avoid the extra dependency given how infrequently this option is specified:

screen shot 2017-02-11 at 21 32 01

- Make openssl mandatory and libidn recommended per discussions in Homebrew#9613;
- Test: write retrieved page to /dev/null (the hundreds of kilobytes of output
  on stdout is useless).
@zmwangx
Copy link
Contributor Author

zmwangx commented Feb 11, 2017

@MikeMcQuaid Internationalized domain name.

"IRI support" is somewhat confusing because Wget can handle Unicode in path, query and fragment (which is simply a matter of percent-encoding) just fine without libidn; however, it requires libidn to support Unicode characters in the domain name.

As I said in #9613 (comment), I'm recommending libidn on principle: 1. Wget defaults to supporting IDNs (when libidn is available, that is); 2. The web is increasingly internationalized; I've certainly encountered IDNs in the wild, and I expect to see more. (I do realize you can translate IDNs elsewhere and feed the ASCII version to a Wget without IDN support, e.g., copying from browser's address bar.)

However, I just realized that Wget 1.19 dropped support for libidn in favor of libidn2 (commit bettermirror/wget@00ae9b4). Therefore, I removed libidn-related code from the formula.

Someone could contribute a libidn2 formula to bring the option back.

@ilovezfs
Copy link
Contributor

Also probably important to note the security concerns with idn: https://en.wikipedia.org/wiki/Internationalized_domain_name#ASCII_spoofing_concerns

The use of Unicode in domain names makes it potentially easier to spoof web sites as the visual representation of an IDN string in a web browser may make a spoof site appear indistinguishable to the legitimate site being spoofed, depending on the font used. For example, Unicode character U+0430, Cyrillic small letter a, can look identical to Unicode character U+0061, Latin small letter a, used in English. As a concrete example, using Cyrillic letters а, е (“Ie”/“Ye”, U+0435, looking essentially identical to Latin letter e), Belarusian-Ukrainian і (U+0456, essentially identical to Latin letter i), р (“Er”, U+0440, essentially identical to Latin letter p), we form the URL wіkіреdіа.org (xn--wkd-8cdx9d7hbd.org in encoded form), which is virtually indistinguishable from the visual representation of the legitimate wikipedia.org (possibly depending on fonts).

@MikeMcQuaid
Copy link
Member

Thanks for your contribution to Homebrew! Without people like you submitting PRs we couldn't run this project. You rock!

@zmwangx zmwangx deleted the wget-1.19.1 branch February 12, 2017 15:34
@jdvorak001 jdvorak001 mentioned this pull request Dec 17, 2017
4 tasks
@Homebrew Homebrew locked and limited conversation to collaborators May 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants