rest-server: cleanup unreadable code

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

rest-server: use BAD_REQUEST for non-GET on file-paths

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

rest-server: make handle_request a method of ApiConfig

This is what actually defines the API server after all.
The ApiService trait in between is a hyper impl detail.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

rest-server: drop allocation in Service impl

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

rest-server: PeerAddr trait, drop proxmox-http dep

We pulled in proxmox-http with the client feature solely to
implement the `Service` trait on
`SslStream<RateLimitedStream<TcpStream>>`.

All those `Service` impls are the same: provide a peer
address and return an `ApiService`.
Let's put the `peer_addr()` call into a trait and build from
there.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

rest-server: make socketpair private

`proxmox_rest_server::socketpair` doesn't make sense as an
external API

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

rest-server: start 0.3 api refactoring

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

ldap: test fixup

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

sys: drop sortable and identity macros

We should not use the sys crate to pull in the sortable
macro, just depend on its crate instead...
And the identity macro used to be required by the sortable
macro, but is not anymore and has been deprecated for a
while, so we can now drop it.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

router: make format&print generic

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

release proxmox-ldap to 0.1.0-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump proxmox-rest-server to 0.2.2-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

rest-server: add handle_worker from backup debug cli

The function has now multiple users, so it is moved
here.

Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>

ldap: fixup d/control

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

ldap: drop Ldap prefix from types that have it

for a bit more consistency and since we tend to repeat stuff
too much

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

fix 'default-features = false' for ldap3

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

ldap: clippy fixups

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

ldap: add debian packaging

Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>

ldap: tests: add LDAP integration tests

This commit adds integration tests to ensure that the crate works as intended.
The tests are executed against a real LDAP server, namely `glauth`. `glauth` was
chosen because it ships as a single, statically compiled binary and can
be configured with a single configuration file.

The tests are written as off-the-shelf unit tests. However, they are
 #[ignored] by default, as they have some special requirements:
   * They required the GLAUTH_BIN environment variable to be set,
     pointing to the location of the `glauth` binary. `glauth` will be
     started and stopped automatically by the test suite.
   * Tests have to be executed sequentially (`--test-threads 1`),
     otherwise multiple instances of the glauth server might bind to the
     same port.

The `run_integration_tests.sh` checks whether GLAUTH_BIN is set, or if
not, attempts to find `glauth` on PATH. The script also ensures that the
tests are run sequentially.

Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>

ldap: allow searching for LDAP entities

This commit adds the search_entities function, which allows to search for
LDAP entities given certain provided criteria.

Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>

ldap: add helpers for constructing LDAP filters

Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>

ldap: add basic user auth functionality

In the LDAP world, authentication is done using the bind operation, where
users are authenticated with the tuple (dn, password). Since we only know
the user's username, it is first necessary to look up the user's
domain (dn).

Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>

ldap: create new `proxmox-ldap` crate

Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>

sys: fixup error types handling

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

sys: cope with unavailable KSM sharing info

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump proxmox-time to 1.1.5-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

re-add proxmox-uuid d/control

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump proxmox-uuid to 1.0.2-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

update d/control files

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

bump proxmox-rest-server to 0.2.1-1

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

bump proxmox-shared-memory to 0.2.3-1

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

bump proxmox-router to 1.3.1-1

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

bump proxmox-schema to 1.3.6-1

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

bump proxmox-sys to 0.4.2-1

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

update nix to 0.26

it's the version currently shipped by bookworm, so let's unify this widely-used
dependency to make bootstrapping easier.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

update d/control

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

re-add epoch_to_rfc3339_utc on wasm target

This was lost in commit 980d6b26df0e886795f5121036eed0a7511e11dd.

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

clippy fix

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

section-config: silence clippy

these two functions don't actually use the `type_name` parameter, but the
interface including custom formatters require it.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

section-config: make ReST dump reproducible

HashMaps are not ordered, so each package build containing a section config
dump would have the documentation ordered randomly.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

section config: fix handling array schema in unknown sections

Mostly relevant when the config is written out again after parsing it
with unknown sections. Previously, with duplicate keys, only the last
value would be saved. Now, duplicate keys are assumed to be part of
an array schema and handled as such.

Because the unknown section parsing does not know if a certain
property does actually have an array schema, it's not possible to
detect duplicate keys for non-array-schema properties, and if a
property with array-schema shows up only once, it will not be saved as
a Value::Array, but a Value::String.

Writing, or to be precise the format_section_content methods, already
handle Value::Array, so don't need to be adapted.

Fixes: 0cd0d16 ("section config: support allowing unknown section types")
Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>

section config: add test for array schema

where duplicate keys are allowed.

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>

deps: bump api-macro to current version

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump proxmox-api-macro to 1.0.4-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

api: support #[default] attribute

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

update/extend README.rst

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

workspace: switch remaining dependencies

while these are (currently) only used by a single member each, having *all*
dependency versions specified in the top level Cargo.toml only makes the whole
process of managing them less error-prone.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

update d/control

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

io: add boxed module for boxed bytes like vec::zeroed...

- proxmox_io::boxed::uninitialized(len) -> Box<[u8]>
  same as vec::uninitialized, but as a box

- proxmox_io::boxed::zeroed(len) -> Box<[u8]>
  same as vec::zeroed, but as a box

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

io: deny unsafe_op_in_unsafe_fn

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

more workspace dependencies

regex was missed in the first pass, and two intra-workspace dev-dependencies as
well.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

switch exclude to workspace in README

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

buildsys: get crate list via cargo metadata in Makefile

so we don't have to keep this in sync manually

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

add bump.sh

for bumping crates in this workspace (it requires cargo-edit being installed).

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

proxmox-time: drop TryFrom use statement

no longer needed with edition 2021

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

update d/control files

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

update outdated workspace dependencies

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

update d/control files

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

define workspace dependencies in workspace

so that we no longer have to (or forget to) bump the version in multiple places.

notable changes:
- outdated versions have been unified
- proxmox-metrics -> proxmox-async no longer uses explicit empty features
  (proxmox-async doesn't provide any anyway)
- proxmox-subscription -> proxmox-http no longer uses explicit default_features
  = false (proxmox-http has an empty default feature anyway)
- missing path dependencies added (mainly proxmox-rest-server)

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

update d/control files

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

inherit shared, external dependencies

noteworthy changes:
- proxmox-http had a default_features_false dep on hyper, which is dropped (the
  default feature is empty anyway)
- hyper, libc, nix, tokio and url versions are unified
- missing (cosmetic) bindgen feature on zstd enabled everywhere

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

move common metadata to workspace

and switch all crates to 2021 edition as well as a unified "authors" value.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

update d/control files

debcargo 2.6 changed some minor details

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

use statement fixup

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

clippy 1.65 fixes

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

proxmox-schema: bump to 1.3.5

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

schema: update to textwrap 0.16

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

bump proxmox-section-config to 1.0.2-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

minor doc fixup

an 'ignore' block assumes rust syntax, a 'text' block should
just be plain text

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

section config: support allowing unknown section types

Similar to commit c9ede1c ("support unknown types in section config")
in pve-common.

Unknown sections are parsed as String-JSON String key-value pairs
without any additional checks and also written as-is.

Suggested-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>

subscription: recognize 'Suspended' status

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump proxmox-section-config to 1.0.1-1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

section-config: expose order

we already expose the raw sections which are sometimes
easier to use, but we don't expose the order at all this way
otherwise

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

section-config: use Vec for section order

We use none of the additional functionality provided by
VecDeque.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

section config: parse additional properties when schema allows it

Additional properties will be parsed according to the default string
schema.

This is relevant for use cases when the full schema is not known for
some reason or another. In particular this allows support for parsing
older/newer versions of configuration files. One example of this is
the proposed proxmox-mail-forward helper binary, which currently
doesn't have access to the PBS API types for dependency reasons and
is only interested in the email field for the root user. If it can
only use a minimal schema with additional_properties set to true, it
will be robust against changes.

Writing already works, because the ObjectSchema's verify_json()
already handles additional_properties correctly and
format_section_content() handles them like all other properties
(method doesn't depend on the schema).

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>

sys: impl AsFd for PTY

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

sys: impl AsFd for PidFd

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump sys to 0.4.1

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

proxmox-http: fix last changelog entry

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

sys: deprecate RawFdNum

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

sys: add From<OwnedFd/Fd> for Fd/OwnedFd temporarily

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

rest-server: update to OwnedFd

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

sys: deprecate Fd, add its methods as module functions

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

sys: deprecate BorrowedFd

std has this now, stable since 1.63

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump edition in rustfmt.toml

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

rest-server: add packaging and bump to 0.2.0

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

cargo: rest-server: set license property

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

Merge remote-tracking branch 'proxmox-rest-merge/master'

split out from proxmox-backup using `git filter-repo` including
history with the following --paths-from-file:

```
proxmox-rest-server
src/api/server.rs
src/server/command_socket.rs
src/server/config.rs
src/server/environment.rs
src/server/formatter.rs
src/server/h2service.rs
src/server/rest.rs
src/server/state.rs
src/tools/compression.rs
src/tools/daemon.rs
src/tools/file_logger.rs
src/worker_task.rs
```

cargo fmt

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

clippy fixes

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

metrics: bump to 0.2.1

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

subscription: bump to 0.3.0

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

update proxmox-router to 1.3.0

no real change for PBS usage - the ApiHandler enum is marked
non_exhaustive now because it has extra values if the new (enabled by
default) "server" feature is enabled.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

update to proxmox-http 0.7

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

subscription: properly forward verification error

when verifying the server response used for offline mirror keys.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

subscription: use lowercase for Display-ing status

subscription: properly alias 'notfound'

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

subscription: conditionalize checks

signed subscription info files should always be checked to catch
attempts of invalid signatures, but the age and serverid checks only
need to apply to "active" files, else the status might switch from a
more meaningful one to "invalid" by accident.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

subscription: add Expired status

this can be returned by the shop when checking an online subscription.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

http: bump to 0.7.0

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>