Hector Martin

So yeah, um, this is not okay. It is not discoverable and could easily leak sensitive information. Auth credentials even, seriously? Also Chrome does this too. And it is preserved across `mv` to another filesystem.https://twitter.com/gynvael/status/1077671412847046657 …

not that having a password in the URL would be a good security design anyway

This is how HTTP Basic authentication works. It's part of the HTTP URI standard. And wget is smart enough to hide the password when showing its messages on stderr, but then goes ahead and dumps it in the xattr verbatim.

You don’t need to support credentials in URLs in order to support basic auth. The credentials are placed in header fields in the HTTP get request.

Credentials in a URI is one (perhaps the only?) standard way for supplying the contents of that header field that is cross-application.

Actually, working with basic auth recently I found that most browsers have already dropped support for URL creds. Too easy to exploit. Wget needs to fix, but one possible fix is dropping URL creds entirely.