summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* [Win32] Fix for USE_WATT32Gisle Vanem2021-06-041-1/+1
| | | My Watt-32 tcp/ip stack works on Windows but it does not have `WSAIoctl()`
* url: bad CURLOPT_CONNECT_TO syntax now returns errorAlexis Vachette2021-06-043-11/+47
| | | | | | Added test 3020 to verify Closes #7183
* github: remove the cmake macOS gcc-8 jobsDaniel Stenberg2021-06-041-3/+0
| | | | | | | They're too similar to the gcc-9 ones to be useful (and seems to not work anymore). Closes #7187
* test269: disable for hyperDaniel Stenberg2021-06-035-2/+19
| | | | | | | --ignore-content-length / CURLOPT_IGNORE_CONTENT_LENGTH doesn't work with hyper. Closes #7184
* runtests: enable 'hyper mode' only for HTTP testsDaniel Stenberg2021-06-033-8/+11
| | | | | | | | | | | The 'hyper mode' makes line-ending checks work in the test suite for when hyper is used. Now it also requires that HTTP or HTTPS are mentioned as keywords to be enabled so that it doesn't wrongly adjusts tests for other protocols. This makes test 271 (TFTP) work again in hyper enabled builds. Closes #7185
* hostip: bad CURLOPT_RESOLVE syntax now returns errorAlexis Vachette2021-06-033-3/+41
| | | | | | Added test 3019 Fixes #7170 Closes #7174
* cookies: fix typo and expand commentDaniel Gustafsson2021-06-031-1/+2
| | | | | Fix a typo in the sorting comment, and while in there elaborate slightly on why creationtime can be used as a tiebreaker.
* cookies: remove unused headerDaniel Gustafsson2021-06-031-1/+0
| | | | | | | | | Commit 1c1d9f1affbd3367bcb24062e261d0ea5d185e3a removed the last use for the inet_pton.h headerfile, this removes the inclusion of the header. Closes: #7182 Reviewed-by: Daniel Stenberg <daniel@haxx.se>
* Revert "socketpair: fix potential hangs"Daniel Stenberg2021-06-031-25/+9
| | | | | | This reverts commit c769d1eab4de8b9f1bd84d992c63692fdc43c5be. See #7144 for details
* socketpair: fix potential hangsPaul Groke2021-06-031-9/+25
| | | | | | | | | | | Fixes potential hang in accept by using select + non-blocking accept. Fixes potential hang in peer check by replacing the send/recv check with a getsockname/getpeername check. Adds length check for returned sockaddr data. Closes #7144
* runtests: parse data/Makefile.inc instead of using makeDaniel Stenberg2021-06-022-9/+9
| | | | | | | | | | | The warning about missing entries in that file then doesn't require that the Makefile has been regenerated which was confusing. The scan for the test num is a little more error prone than before (since now it doesn't actually verify that it is legitimate Makefile syntax), but I think it is good enough. Closes #7177
* filecheck: quietly remove test-place/*~Harry Sintonen2021-06-021-1/+1
| | | | Closes #7179
* CURLE_SETOPT_OPTION_SYNTAX: new error name for wrong setopt syntaxDaniel Stenberg2021-06-026-8/+13
| | | | | | | | | For options that pass in lists or strings that are subsequently parsed and must be correct. This broadens the scope for the option previously known as CURLE_TELNET_OPTION_SYNTAX but the old name is of course still provided as a #define for existing applications. Closes #7175
* tests: fix Accept-Encoding strips to work with Hyper buildsDaniel Stenberg2021-06-0214-39/+32
| | | | | | | | | | The previous strip also removed the CR which turned problematic. valgrind.supp: add zstd suppression using hyper Reported-and-analyzed-by: Kevin Burke Fixes #7169 Closes #7171
* github: timeout jobs on macOS after 90 minutesDaniel Stenberg2021-06-021-0/+1
| | | | | Assisted-by: Marc Hoersken Closes #7173
* mqtt: detect illegal and too large file sizeHarry Sintonen2021-06-025-4/+164
| | | | | Add test 3017 and 3018 to verify. Closes #7166
* cmake: add CURL_DISABLE_NTLM optiontheawless2021-06-021-4/+6
| | | | Closes #7028
* configure: add --disable-ntlm optiontheawless2021-06-021-1/+21
| | | | Closes #7028
* define: re-add CURL_DISABLE_NTLM and corresponding ifdefstheawless2021-06-024-17/+26
| | | | | | | This flag will be further exposed by adding build options. Reverts #6809 Closes #7028
* RELEASE-NOTES: syncedDaniel Stenberg2021-06-021-5/+55
|
* travis: delete --enable-hsts option (it is the default now) [ci skip]Viktor Szakats2021-06-011-1/+1
| | | | | Reviewed-by: Daniel Stenberg Closes #7167
* hostip: fix 3 coverity complaintsDaniel Stenberg2021-06-011-3/+6
| | | | | | | | | | | | | | Follow-up to 1a0ebf6632f889eed - Check the return code to Curl_inet_pton() in two instances, even though we know the input is valid so the functions won't fail. - Clear the 'struct sockaddr_in' struct before use so that the 'sin_zero' field isn't left uninitialized. Detected by Coverity. Assisted-by: Harry Sintonen Closes #7163
* c-hyper: fix NTLM on closed connection tested with test159Daniel Stenberg2021-06-012-2/+21
| | | | Closes #7154
* conncache: lowercase the hash key for better matchDaniel Stenberg2021-06-011-0/+2
| | | | | | | | | | As host names are case insensitive, the use of case sensitive hashing caused unnecesary cache misses and therefore lost performance. This lowercases the hash key. Reported-by: Harry Sintonen Fixes #7159 Closes #7161
* mbedtls: make mbedtls_strerror always workDaniel Stenberg2021-06-011-30/+6
| | | | | | | If the function doesn't exist, provide a macro that just clears the error message. Removes #ifdef uses from the code. Closes #7162
* vtls: exit addsessionid if no cache is initedDaniel Stenberg2021-06-011-2/+7
| | | | | | | | Follow-up to b249592d29ae0 Avoids NULL pointer derefs. Closes #7165
* Curl_ntlm_core_mk_nt_hash: fix OOM in error pathHarry Sintonen2021-06-011-9/+6
| | | | Closes #7164
* ssl: read pending close notify alert before closing the connectionMichael Kaufmann2021-06-015-1/+28
| | | | | | | | | | | | | | | | | | | | | This avoids a TCP reset (RST) if the server initiates a connection shutdown by sending an SSL close notify alert and then closes the TCP connection. For SSL connections, usually the server announces that it will close the connection with an SSL close notify alert. curl should read this alert. If curl does not read this alert and just closes the connection, some operating systems close the TCP connection with an RST flag. See RFC 1122, section 4.2.2.13 If curl reads the close notify alert, the TCP connection is closed normally with a FIN flag. The new code is similar to existing code in the "SSL shutdown" function: try to read an alert (non-blocking), and ignore any read errors. Closes #7095
* setopt: fix incorrect commentsLaurent Dufresne2021-06-011-4/+4
| | | | Closes #7157
* mbedtls: add support for cert and key blob optionsLaurent Dufresne2021-06-011-12/+50
| | | | | | | CURLOPT_SSLCERT_BLOB and CURLOPT_SSLKEY_BLOB weren't usable with mbedtls backend, so the support was added. Closes #7157
* cmake: try well-known send/recv signature for AppleGregor Jasny2021-06-011-64/+78
| | | | | | | | | | | | | | | | | | | | | | | | The CMake `try_compile` command is especially slow for the Xcode generator. With this patch applied it first tests for the currently used (and Open Group specified) send/recv signature. In case this fails testing falls-back to the permutations. speed-up: ``` time cmake .. -GNinja -DCMAKE_USE_SECTRANSP=ON -DHTTP_ONLY=ON -DCMAKE_USE_LIBSSH2=OFF before: 11.64s user 11.09s system 55% cpu 40.754 total after: 7.84s user 6.57s system 51% cpu 28.074 total ``` ``` time cmake .. -GXcode -DCMAKE_USE_SECTRANSP=ON -DHTTP_ONLY=ON -DCMAKE_USE_LIBSSH2=OFF before: 217.07s user 104.15s system 60% cpu 8:51.79 total after: 108.76s user 51.80s system 58% cpu 4:32.58 total ``` Closes #7158
* http2: init recvbuf struct for pushed streamsDaniel Stenberg2021-05-311-0/+2
| | | | | | | | Debug builds would warn that these structs were not initialized properly for pushed streams. Ref: #7148 Closes #7153
* Curl_ssl_getsessionid: fail if no session cache existsDaniel Stenberg2021-05-311-2/+3
| | | | | | | | | This function might get called for an easy handle for which the session cache hasn't been setup. It now just returns a "miss" in that case. Reported-by: Christoph M. Becker Fixes #7148 Closes #7153
* GOVERNANCE: add 'user', 'committer' and 'contributor'Daniel Stenberg2021-05-311-7/+22
| | | | | | As those are commonly used terms in the project. Closes #7151
* URL-SYNTAX.md: document the new 'localhost' treatmentDaniel Stenberg2021-05-311-0/+9
|
* hostip: make 'localhost' return fixed valuesDaniel Stenberg2021-05-311-2/+73
| | | | | | | | | | | | Resolving the case insensitive host name 'localhost' now returns the addresses 127.0.0.1 and (if IPv6 is enabled) ::1 without using any resolver. This removes the risk that users accidentally resolves 'localhost' to something else. By making sure 'localhost' is always local, we can assume a "secure context" for such transfers (for cookies etc). Closes #7039
* docs: fix typosDaniel Gustafsson2021-05-312-4/+4
|
* hsts: ignore numberical IP address hostsDaniel Stenberg2021-05-306-40/+32
| | | | | | | | | Also, use a single function library-wide for detecting if a given hostname is a numerical IP address. Reported-by: Harry Sintonen Fixes #7146 Closes #7149
* test178: adjust for hyperDaniel Stenberg2021-05-291-0/+10
| | | | | | | | | | | Hyper returns the same error for wrong HTTP version as for negative content-length. Test 178 verifies that negative content-length is rejected but the hyper backend will return a different error for it (and without any helpful message telling why the message was bad). It will also not return any headers at all for the response, not even the ones that arrived before the error. Closes #7147
* HYPER: remove mentions of deprecated development branchDaniel Stenberg2021-05-281-3/+1
|
* c-hyper: handle NULL from hyper_buf_copy()Daniel Stenberg2021-05-281-4/+13
| | | | Closes #7143
* HSTS: not experimental anymoreDaniel Stenberg2021-05-281-2/+2
|
* INSTALL: use correct extension for CURL-DISABLE.mdDouglas R. Reno2021-05-271-1/+1
| | | | | | | | In INSTALL.MD, it's currently set to CURL-DISABLE-md instead of CURL-DISABLE.md. This generates a 404 on the cURL website as well as when viewing the docs through Github. Closes #7142
* travis: run tests 1 - 153 with hyperDaniel Stenberg2021-05-271-1/+1
|
* c-hyper: convert HYPERE_INVALID_PEER_MESSAGE to CURLE_UNSUPPORTED_PROTOCOLDaniel Stenberg2021-05-271-0/+2
| | | | | | Makes test 129 work (HTTP/1.2 response). Closes #7141
* http_proxy: deal with non-200 CONNECT response with HyperDaniel Stenberg2021-05-271-3/+32
| | | | | | Makes test 94 and 95 work Closes #7141
* c-hyper: clear NTLM auth buffer when request is issuedDaniel Stenberg2021-05-272-1/+5
| | | | | | | | | To prevent previous ones to get reused on subsequent requests. Matches how the built-in HTTP code works. Makes test 90 to 93 work. Add test 90 to 93 in travis. Closes #7139
* schannel: set ALPN length correctly for HTTP/2Joel Depooter2021-05-271-0/+1
| | | | | | | | | | In a3268eca792f1 this code was changed to use the ALPN_H2 constant instead of the NGHTTP2_PROTO_ALPN constant. However, these constants are not the same. The nghttp2 constant included the length of the string, like this: "\x2h2". The ALPN_H2 constant is just "h2". Therefore we need to re-add the length of the string to the ALPN buffer. Closes #7138
* travis: run tests 1-89 in the hyper buildDaniel Stenberg2021-05-271-1/+1
| | | | Closes #7137
* Revert "c-hyper: handle body on HYPER_TASK_EMPTY"Daniel Stenberg2021-05-271-1/+1
| | | | | | | | This reverts commit c3eefa95c31f55657f0af422e8268d738f689066. Reported-by: Kevin Burke Fixes #7122 Closes #7136