Skip to content

--netrc-file with non-existent / non-readable file - no error message #16163

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.

Sign up for GitHub

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

Jump to bottom
Closed
arlt opened this issue Feb 4, 2025 · 0 comments
Closed

--netrc-file with non-existent / non-readable file - no error message #16163

arlt opened this issue Feb 4, 2025 · 0 comments
Assignees
@bagder

Comments

@arlt
Copy link

arlt commented Feb 4, 2025

I did this

curl --location https://fqdn/file --netrc-file which_does_not_exist doesn't produce an error message / is silently ignored

I expected the following

curl output:

curl: netrc-file "which_does_not_exist" is not readable or does not exist

curl/libcurl version

curl 7.76.1 (x86_64-redhat-linux-gnu) libcurl/7.76.1 OpenSSL/3.2.2 zlib/1.2.11 brotli/1.0.9 libidn2/2.3.0 libpsl/0.21.1 (+libidn2/2.3.0) libssh/0.10.4/openssl/zlib nghttp2/1.43.0
Release-Date: 2021-04-14
Protocols: dict file ftp ftps gopher gophers http https imap imaps ldap ldaps mqtt pop3 pop3s rtsp scp sftp smb smbs smtp smtps telnet tftp
Features: alt-svc AsynchDNS brotli GSS-API HTTP2 HTTPS-proxy IDN IPv6 Kerberos Largefile libz NTLM NTLM_WB PSL SPNEGO SSL TLS-SRP UnixSockets

operating system

Linux myvm 5.14.0-503.22.1.el9_5.x86_64 #1 SMP PREEMPT_DYNAMIC Wed Jan 15 08:02:15 EST 2025 x86_64 x86_64 x86_64 GNU/Linux
@bagder bagder self-assigned this Feb 4, 2025
bagder added a commit that referenced this issue Feb 4, 2025
@bagder
netrc: return code cleanup, fix missing file error

Verified

This commit was signed with the committer’s verified signature.
bagder Daniel Stenberg
GPG key ID: 5CC908FDB71E12C2
Verified
Learn about vigilant mode
Loading
Loading status checks…
e629198 
Made the netrc parser return a more consistent set of error codes and
error messages, and also return error properly if the .netrc file is
missing.

Add test 697 to verify error on missing netrc file.

Fixes #16163
Reported-by: arlt on github
@bagder bagder mentioned this issue Feb 4, 2025
Closed
@bagder bagder closed this as completed in 943de21 Feb 6, 2025
@gepbird gepbird mentioned this issue Mar 12, 2025
Closed
3 tasks
hubot pushed a commit to aosp-mirror/tools_repo that referenced this issue Apr 10, 2025
project: use --netrc-optional instead of --netrc
97dc5c1 
Some users are reporting a "curl: (26) .netrc error: no such file"
message on sync caused by an change to curl behavior.
See curl/curl#16163.

Use --netrc-optional which was introduced in curl version 7.9.8
released in 2002.

Bug: 409354839
Change-Id: I8365c6e806968a4ee765a7e023b4bced30489c20
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/467026
Reviewed-by: Mike Frysinger <vapier@google.com>
Commit-Queue: Gavin Mak <gavinmak@google.com>
Tested-by: Gavin Mak <gavinmak@google.com>
Reviewed-by: Josip Sokcevic <sokcevic@chromium.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bagder bagder

Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

netrc: return code cleanup, fix missing file error curl/curl
2 participants
@bagder @arlt