url: ensure minimum components for formatting

url: ensure minimum components for formatting

Merge pull request #6026 from libgit2/ethomson/proxy

Update proxy configuration

remote: refactor proxy detection

Update the proxy detection for a remote.

1. Honor `http.<url>.proxy` syntax for a remote's direct URL and
   parent URLs.
2. Honor an empty configuration URL to override a proxy configuration.

Add tests to ensure that configuration specificity is honored.

url: introduce `git_net_url_dup`

url: introduce `git_net_url_matches_pattern_list`

Provide a utility method on a url to determine if it matches any pattern
in a comma-separated list, similar to what one would find in `NO_PROXY`
environment variables.

url: introduce `git_net_url_matches_pattern`

Provide a method to determine if a given URL matches a host:port pattern
like the ones found in `NO_PROXY` environment variables.

tests: create the network::url test hierarcy

util: introduce `git__strlcmp`

Introduce a utility function that compares a NUL terminated string to a
possibly not-NUL terminated string with length.  This is similar to
`strncmp` but with an added check to ensure that the lengths match (not
just the `size` portion of the two strings).

Add NO_PROXY env support

Item 2 of 3 from #4164

Signed-off-by: Mathieu Parent <math.parent@gmail.com>

Merge pull request #6024 from lhchavez/fix-possible-null-pointer-dereference

Fix a possible null pointer dereference

Fix a possible null pointer dereference

This change fixes a possible null pointer dereference if anything inside
`git_commit_graph_writer_add_index_file` fails before the packfile being
valid.

https://scan6.coverity.com/reports.htm#v52218/p10377/fileInstanceId=122935896&defectInstanceId=32525576&mergedDefectId=1461634

v1.2: update version information

v1.2: changelog

Merge pull request #5943 from kcsaul/fix/5851

Fix worktree iteration when repository has no common directory

Merge pull request #6022 from lollipopman/connect-proxy-host-header

Set Host Header to match CONNECT authority target

Merge branch 'cgraph-write' into main

commit graph: formatting fixes

Set Host Header to match CONNECT authority target

Prior to this change, for CONNECT requests, the Host header was set to
the host and port of the target http proxy. However, per the rfc7230 for
HTTP/1.1 this is incorrect as the Host header should match the target of
the CONNECT request, as detailed in section 5.3.3 & 5.4.

  5.3.3.  authority-form

   The authority-form of request-target is only used for CONNECT
   requests (Section 4.3.6 of [RFC7231]).

     authority-form = authority

   When making a CONNECT request to establish a tunnel through one or
   more proxies, a client MUST send only the target URI's authority
   component (excluding any userinfo and its "@" delimiter) as the
   request-target.  For example,

     CONNECT www.example.com:80 HTTP/1.1

  5.4.  Host

   <snip>

   A client MUST send a Host header field in all HTTP/1.1 request
   messages.  If the target URI includes an authority component, then a
   client MUST send a field-value for Host that is identical to that
   authority component, excluding any userinfo subcomponent and its "@"
   delimiter (Section 2.7.1).  If the authority component is missing or
   undefined for the target URI, then a client MUST send a Host header
   field with an empty field-value.

This issue was noticed when proxying requests through HAProxy 2.2 which
rejects these invalid http requests.

Merge pull request #6018 from libgit2/ethomson/fixups

Fixes from code analysis

date: promote before multiply

filter: don't overwrite error value

hashsig: close fd on error

error: don't check string after assert

pack: don't assert in the lock

midx: return an error when detected

httpclient: actually return `GIT_EAUTH`

Merge pull request #6017 from libgit2/ethomson/buf_is_readonly

buf: deprecate public git_buf writing functions

Merge pull request #6011 from libgit2/ethomson/filter_apply

filter: filter drivers stop taking git_buf as user input

Merge pull request #6016 from libgit2/ethomson/commit_create_cb

Introduce `create_commit_cb`, deprecate `signing_cb`

Merge pull request #6012 from libgit2/ethomson/custom_url

remote: introduce remote_ready_cb, deprecate resolve_url callback

Merge pull request #6009 from libgit2/ethomson/custom_cert_locations

opts: test GIT_OPT_SET_SSL_CERT_LOCATIONS

mbedTLS: Fix setting certificate directory

fixes #6003

opts: test GIT_OPT_SET_SSL_CERT_LOCATIONS

Include a self-signed certificate for test.libgit2.org:1443 that we can
use to verify that GIT_OPT_SET_SSL_CERT_LOCATIONS works.

Merge remote-tracking branch 'origin/main' into cgraph-write

Merge pull request #5841 from J0Nes90/features/checkout-dry-run

Checkout dry-run

Update include/git2/checkout.h

Merge pull request #5405 from lhchavez/multi-pack-index-odb-write

midx: Introduce git_odb_write_multi_pack_index()

Merge pull request #5395 from josharian/http-use-eauth

Use error code GIT_EAUTH for authentication failures

Merge branch 'main' into multi-pack-index-odb-write

Merge pull request #5404 from lhchavez/multi-pack-index-write

midx: Add a way to write multi-pack-index files

Merge branch 'main' into http-use-eauth

ci: stop on test failure

buf: deprecate public git_buf writing functions

A `git_buf` is now a read-only structure as far as callers are
concerned.  This is a mechanism that we can return data to callers
using memory that is owned by the library and can be cleaned up by
callers (using `git_buf_dispose`).

A `git_buf` can no longer be allocated by callers or provided to the
library.

filter: deprecate apply function

remote: deprecate resolve_url callback

Using a callback to set a resolve_url is not particularly idiomatic.
Deprecate it in favor of the `set_instance_url` and
`set_instance_pushurl` functions which can now be called from the
`git_remote_ready_cb` callback.

remote: introduce git_remote_ready_cb

Introduce a new callback that fires when the remote is ready to connect.

rebase: deprecate signing_cb

The signing callback should not be used; instead, callers should provide
a commit_create_cb, perform the signing and commit creation themselves.

rebase: introduce git_commit_create_cb

Introduce a new mechanism for `git_rebase_commit` for callers to
customize the experience.  Instead of assuming that we produce the
commit for them, provide a commit creation callback that allows callers
to produce the commit themselves and return the resulting commit id.

Merge pull request #6015 from boretrk/c89-comments

tests: change comments to c89 style

Merge pull request #5508 from libgit2/ethomson/devcontainer

WIP: .devcontainer: settings for a codespace workflow

Merge pull request #6013 from lolgear/cmake_hash_sha_missing_header

CMake. hash sha1 header has been added.

cmake: select hashes umbrella variable has been removed.

Co-authored-by: Edward Thomson <ethomson@github.com>

cmake: select hashes missing umbrella header has been added.

filter: use streaming filters in tests

ident: use streaming filters

crlf: use streaming filters

filter: proxy_stream is now git_filter_buffered_stream

The filter's proxy_stream is used to adapt filters that only provided an
`apply` function into a `stream` function.  Make this internal to the
library instead of private to the filter file.  This will allow the
filters to use it directly, instead of relying on the filter
functionality to do the proxying.

filter: add docs for `git_filter_stream_fn`

remote: introduce set_instance_url

Users may want to override the URL on a particular instance of a remote,
instead of updating the configuration.  Previously, users could use a
callback to do this, but this is not particularly idiomatic.

Merge pull request #6008 from boretrk/array

git_array_alloc: return objects of correct type

Merge pull request #5747 from lhchavez/atomic-tests

Homogenize semantics for atomic-related functions

midx: Introduce git_odb_write_multi_pack_index()

This change introduces git_odb_write_multi_pack_index(), which creates a
`multi-pack-index` file from all the `.pack` files that have been loaded
in the ODB.

Fixes: #5399

midx: Add a way to write multi-pack-index files

This change adds the git_midx_writer_* functions to allow to
write and create `multi-pack-index` files from `.idx`/`.pack` files.

Part of: #5399

Merge remote-tracking branch 'origin/main' into multi-pack-index-write

Review feedback

Merge pull request #6010 from A-Ovchinnikov-mx/a-ovchin/remote-head-branch-clone

Set refs/remotes/origin/HEAD to default branch when branch is specified

Fixups for the latest changes in the array interface

Merge remote-tracking branch 'origin/main' into cgraph-write

Homogenize semantics for atomic-related functions

There were some subtle semantic differences between the various
implementations of atomic functions. Now they behave the same, have
tests and are better documented to avoid this from happening again in
the future.

Of note:

* The semantics chosen for `git_atomic_compare_and_swap` match
  `InterlockedCompareExchangePointer`/`__sync_cal_compare_and_swap` now.
* The semantics chosen for `git_atomic_add` match
  `InterlockedAdd`/`__atomic_add_fetch`.
* `git_atomic_swap` and `git_atomic_load` still have a bit of semantic
  difference with the gcc builtins / msvc interlocked operations, since
  they require an l-value (not a pointer). If desired, this can be
  homogenized.

Make the defaultable fields defaultable

Also, add `git_commit_graph_writer_options_init`!

Check if default branch matches refspec

clone: set refs/remotes/origin/HEAD to default branch when branch is specified, attempt 2

Revert "Merge pull request #5775 from libgit2/ethomson/clone_branch"

This reverts commit 487f2a8287c4d1d41d39e05ac53d0e50c679d7e9, reversing
changes made to c6cf7f0e6927f1b264a7dda2467d70b9f744a01f.

git_array_alloc: return objects of correct type

Merge pull request #6007 from boretrk/array

array: check dereference from void * type

Merge pull request #6006 from boretrk/c11-warnings

GCC C11 warnings

array: fix dereference from void * type

array: make gcc keep the type of the NULL return value

win32: name the dummy union in GIT_REPARSE_DATA_BUFFER

Instead of buf->"typeofbuffer"ReparseBuffer the members will be
referenced with buf->ReparseBuffer."typeofbuffer"

https://docs.microsoft.com/en-us/windows-hardware/drivers/ddi/ntifs/ns-ntifs-_reparse_data_buffer?redirectedfrom=MSDN
calls the union DUMMYUNIONNAME but that looks a bit cluttered.

Merge pull request #5802 from lhchavez/git-warn-unused-result

Introduce GIT_WARN_UNUSED_RESULT

Merge pull request #5950 from boretrk/posixtest

open: input validation for empty segments in path

enable gcc warnings for C11 specific extensions

Merge pull request #6005 from boretrk/c11-warnings

C11 warnings

c90/c99: name the unnamed union in git_hash_ctx

enable warnings for C11 specific extensions

Merge pull request #5974 from libgit2/ethomson/dlopen_ssl

Dynamically load OpenSSL (optionally)

ci: run dynamically loaded openssl workflows nightly

Add Xenial, Bionic, CentOS 7 and 8 workflows with OpenSSL-Dynamic builds
nightly.

openssl: lazily load libraries when dynamically loading

Defer dlopen until it's needed when dynamically loading OpenSSL
libraries.

openssl: dynamically load on macOS

tests: show errors when libgit2_init fails

ntlmclient: update to ntlmclient 0.9.1

The ntlmclient dependency can now dynamically load OpenSSL.

valgrind: suppress leaks in dlopen and newer libraries

dlopen sets up some thread-local state that isn't cleaned up by
`dlclose`.  Additionally, now that we're linking against different
versions of libssh2 and OpenSSL, we're seeing different leak signatures.

openssl: dynamically load libssl and symbols (optionally)

Provide an interface around OpenSSL to dynamically load the libraries
and symbols, so that users can distribute a libgit2 library that is not
linked directly against OpenSSL.  This enables users to target multiple
distributions with a single binary.

This mechanism is optional and disabled by default.  Configure cmake
with -DUSE_HTTPS=OpenSSL-Dynamic to use it.

openssl: separate legacy api

Refactor the OpenSSL stream implementation so that the legacy code is better
abstracted.  This will enable future development.

Merge pull request #6000 from libgit2/ethomson/ci

ci: tag new containers with the latest tag

ci: update container versions

Update all the container versions to force a rebuild so that they'll get
tagged with latest (due to changes in the CI scripts).

ci: tag new containers with the latest tag

Merge pull request #5995 from libgit2/ethomson/centos_ci

ci: update centos builds