summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* conncache: various concept cleanupsbagder/conncache-cleanupDaniel Stenberg2020-04-2710-127/+139
| | | | | | | | | | | | | | | | More connection cache accesses are protected by locks. CONNCACHE_* is a beter prefix for the connection cache lock macros. Curl_attach_connnection: now called as soon as there's a connection struct available and before the connection is added to the connection cache. Curl_disconnect: now assumes that the connection is already removed from the connection cache. Ref: #4915 Closes #5009
* copyright updates: adjust year rangesDaniel Stenberg2020-04-263-3/+3
|
* CI: do not include */ci branches in PR buildsMarc Hoersken2020-04-261-1/+0
| | | | Align Azure Pipelines with GitHub Actions.
* runtests: check for the disabled tests relative srcdirDaniel Stenberg2020-04-251-1/+1
| | | | | | | | | | To make it work correctly for out-of-tree builds. Follow-up to 75e8feb6fb08b Bug: https://github.com/curl/curl/pull/5288#issuecomment-619346389 Reported-by: Marcel Raad Closes #5297
* runtests: revert commenting out a line I did for debuggingDaniel Stenberg2020-04-251-1/+1
| | | | Follow-up to 11091cd4d. It was not meant to be pushed!
* smtp: set auth correctlyDaniel Stenberg2020-04-251-4/+1
| | | | | | | | | | | | Regression since 7.69.0 and 68fb25fa3fcff. The code wrongly assigned 'from' instead of 'auth' which probably was a copy and paste mistake from other code, leading to that auth could remain NULL and later cause an error to be returned. Assisted-by: Eric Sauvageau Fixes #5294 Closes #5295
* lib: clean up whitespaceMarcel Raad2020-04-253-3/+0
| | | | This fixes CodeFactor warnings.
* libssh: avoid options override by configuration filesAnderson Toshiyuki Sasaki2020-04-251-14/+47
| | | | | | | | | | | | | | | Previously, options set explicitly through command line options could be overridden by the configuration files parsed automatically when ssh_connect() was called. By calling ssh_options_parse_config() explicitly, the configuration files are parsed before setting the options, avoiding the options override. Once the configuration files are parsed, the automatic configuration parsing is not executed. Fixes #4972 Closes #5283 Signed-off-by: Anderson Toshiyuki Sasaki <ansasaki@redhat.com>
* runtests: when <killserver> mentions http, kill http/2 tooDaniel Stenberg2020-04-251-57/+9
| | | | | | | | | | | Since the http2 test server is a mere proxy that needs to know about the dynamic port the HTTP server is using, it too needs to get restarted when the http server is killed. A regression caused by 80d6515. Fixes #5289 Closes #5291
* docs: fix two typosYuri Slobodyanyuk2020-04-252-3/+3
| | | | Closes #5292
* tests/git: ignore mqttd and port filesEmil Engler2020-04-242-0/+2
| | | | Closes #5290
* tests: make runtests check that disabled tests existsDaniel Stenberg2020-04-242-2/+7
| | | | | | | ... and error out if so. Removed '536' from DISABLED as there is no such test file. Closes #5288
* test1154: set a proper nameDaniel Stenberg2020-04-241-1/+1
|
* select: make Curl_socket_check take timediff_t timeoutDaniel Stenberg2020-04-235-11/+11
| | | | | | | | | | Coverity found CID 1461718: Integer handling issues (CONSTANT_EXPRESSION_RESULT) "timeout_ms > 9223372036854775807L" is always false regardless of the values of its operands. This occurs as the logical second operand of "||". Closes #5240
* libcurl-multi.3: added missing full stopi-ky2020-04-231-1/+1
| | | | Closes #5285
* transfer: Switch PUT to GET/HEAD on 303 redirectJay Satiro2020-04-223-7/+91
| | | | | | | | | | | | | | Prior to this change if there was a 303 reply to a PUT request then the subsequent request to respond to that redirect would also be a PUT. It was determined that was most likely incorrect based on the language of the RFCs. Basically 303 means "see other" resource, which implies it is most likely not the same resource, therefore we should not try to PUT to that different resource. Refer to the discussions in #5237 and #5248 for more information. Fixes https://github.com/curl/curl/issues/5237 Closes https://github.com/curl/curl/pull/5248
* lib/mk-ca-bundle: skip empty certsDaniel Stenberg2020-04-221-1/+6
| | | | | | | Reviewed-by: Emil Engler Reported-by: Ashwin Metpalli Fixes #5278 Closes #5280
* version: skip idn2_check_version() check and add precautionDaniel Stenberg2020-04-221-9/+11
| | | | | | A gcc-10's -fanalyze complaint made me spot and do these improvements. Closes #5281
* RELEASE-NOTES: syncedDaniel Stenberg2020-04-221-6/+27
|
* curl.h: update comment typoBrian Bergeron2020-04-221-1/+1
| | | | | | "routines with be invoked" -> "routines will be invoked" Closes #5279
* GnuTLS: Don't skip really long certificate fieldsEmil Engler2020-04-212-19/+15
| | | | Closes #5271
* gnutls: bump lowest supported version to 3.1.10Daniel Stenberg2020-04-213-290/+53
| | | | | | | | | GnuTLS 3.1.10 added new functions we want to use. That version was released on Mar 22, 2013. Removing support for older versions also greatly simplifies the code. Ref: #5271 Closes #5276
* mqtt: make NOSTATE get within the debug name arrayDaniel Stenberg2020-04-202-2/+4
|
* tests: run the RTSP test server on a dynamic port numberDaniel Stenberg2020-04-203-21/+77
| | | | | | To avoid port collisions. Closes #5272
* tests: add %NOLISTENPORT and use itDaniel Stenberg2020-04-208-6/+9
| | | | | | | | | | | | The purpose with this variable is to provide a port number that is reasonably likely to not have a listener on the local host so that tests can try connect failures against it. It uses port 47 - "reserved" according to IANA. Updated six tests to use it instead of the previous different ports. Assisted-by: Emil Engler Closes #5270
* mqtt: remove code with no purposeDaniel Stenberg2020-04-201-2/+0
| | | | | | | | | | Detected by Coverity. CID 1462319. "The same code is executed when the condition result is true or false, because the code in the if-then branch and after the if statement is identical." Closes #5275
* mqtt: fix Curl_read() error handling while reading remaining lengthDaniel Stenberg2020-04-201-0/+2
| | | | | | Detected by Coverity. CID 1462320. Closes #5274
* server/tftpd: fix compiler warningDaniel Stenberg2020-04-201-1/+1
| | | | | Follow-up from 369ce38ac1d Reported-by: Marc Hörsken
* http: free memory when Alt-Used header creation fails due to OOMDaniel Stenberg2020-04-201-8/+5
| | | | | | Reported-by: James Fuller Fixes #5268 Closes #5269
* lib: fix typos in comments and errormessagesDaniel Gustafsson2020-04-203-5/+5
| | | | | This fixes a few randomly spotted typos in recently merged code, most notably one in a userfacing errormessage the schannel code.
* tests: run the SOCKS test server on a dynamic port numberDaniel Stenberg2020-04-202-12/+21
| | | | Closes #5266
* multi-ssl: reset the SSL backend on `Curl_global_cleanup()`Johannes Schindelin2020-04-201-0/+6
| | | | | | | | | | | | | | | | | | When cURL is compiled with support for multiple SSL backends, it is possible to configure an SSL backend via `curl_global_sslset()`, but only *before* `curl_global_init()` was called. If another SSL backend should be used after that, a user might be tempted to call `curl_global_cleanup()` to start over. However, we did not foresee that use case and forgot to reset the SSL backend in that cleanup. Let's allow that use case. Fixes #5255 Closes #5257 Reported-by: davidedec on github Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
* tests: run the TFTP test server on a dynamic port numberDaniel Stenberg2020-04-203-23/+85
| | | | | | | Picking a dynamic unused port is better than a fixed to avoid the collision risk. Closes #5265
* mqtt: improve the state machineDaniel Stenberg2020-04-2011-99/+402
| | | | | | | | | | To handle PUBLISH before SUBACK and more. Updated the existing tests and added three new ones. Reported-by: Christoph Krey Bug: https://curl.haxx.se/mail/lib-2020-04/0021.html Closes #5246
* runtests: always put test number in servercmd fileDaniel Stenberg2020-04-202-8/+8
|
* RELEASE-NOTES: syncedDaniel Stenberg2020-04-201-12/+36
|
* release-notes.pl: fix parsing typoDaniel Stenberg2020-04-201-2/+2
|
* ensure all references to ports are replaced by varsxquery2020-04-202-6/+6
|
* add more alt-svc test coveragexquery2020-04-203-1/+190
|
* test1247: use http server to get the port number setDaniel Stenberg2020-04-201-1/+1
| | | | Follow-up to 0f5db7b263f
* runtests: use a unix domain socket path with the pid in the nameDaniel Stenberg2020-04-191-1/+1
| | | | | | | To make it impossible for test cases to access the file name without using the proper variable for the purpose. Closes #5264
* src: Remove C99 constructs to ensure C89 complianceTom2020-04-192-4/+6
| | | | | | | | | This fixes the error: 'for' loop initial declaration used outside C99 mode by declaring the loop increment variable in the beginning of the block instead of inside the for loop. Fixes #5254 Reviewed-by: Daniel Gustafsson <daniel@yesql.se>
* runtests: dummy init the ports variables to avoid warningsDaniel Stenberg2020-04-191-30/+32
| | | | ... and generate something that can help debug test cases.
* mime: properly check Content-Type even if it has parametersPatrick Monnerat2020-04-193-3/+84
| | | | | | | | New test 669 checks this fix is effective. Fixes #5256 Closes #5258 Reported-by: thanhchungbtc on github
* tests/FILEFORMAT: converted to markdown and extendedDaniel Stenberg2020-04-194-515/+478
| | | | Closes #5261
* test1245: make it work with dynamic FTP server portbagder/tests-on-dynportDaniel Stenberg2020-04-181-5/+1
|
* test1055: make it work with dynamic FTP portDaniel Stenberg2020-04-181-5/+1
|
* test1028: make it run on dynamic FTP server portDaniel Stenberg2020-04-181-5/+1
|
* tests: move pingpong server to dynamic listening portDaniel Stenberg2020-04-183-62/+92
| | | | | | | | FTP, IMAP, POP3, SMTP and their IPv6 versions are now all on dynamic ports Test 842-845 are unfortunately a bit hard to move over to this concept right now and require "default port" still...
* test1056: work with dynamic HTTP ipv6 portDaniel Stenberg2020-04-181-6/+2
|
View Lorry Controller Status