a lot more clippy lint fixes

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

various clippy lint fixes

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

websocket feature dependency fixup

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

proxmox/tools/websocket: add WebSocket implementation

uses the existing WebSocketReader and Writer to establish a
two-way communication between an upstream and downstream connection.

The upstream connection sends and receives WebSocket frames, while
the downstream one only receives and sends raw data.

For now we do not support extensions, and only accept the protocol version 13

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

proxmox/tools/websocket: replace CallBack with a channel

instead of having a callback that we call on a control frame,
use a channel to send the data to a receiver

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

proxmox/tools/websocket: implement send_control_frame for writer

so that we can easily send a control frame to the endpoint

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

proxmox/tools/websocket: improve mask_bytes and create_frame interface

by using a Box<[u8]> instead of a vector (we do not need it)

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

proxmox/tools/websocket: use io::Error and Result explicitely

and add a helper struct for the ReadResult
(so that the types are properly documented)

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

proxmox/tools/websocket: correctly return eof

only return Ok(0) when the upstream reader did return that, not when
we have no data in the buffer, else the downstream reader believes
we are EOF (even if we are not)

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

proxmox/tools/websocket: use ready macro for WebSocketWriter

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

proxmox/tools/byte_buffer: impl Default

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

proxmox/tools/byte_buffer: improve ByteBuffer interface

by implementing Deref and DerefMut, renaming consume to 'remove_data'
adapt the usage inside of websocket (we did not use it anywhere else for now)

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

tools::common_regex: add IPRE_BRACKET

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

bump proxmox version to 0.2.0-1

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

bump api macro to 0.2.0-1

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

api-macro: test using return schemas within return objects

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

api-macro: reference consistency

API_RETURN_* and API_PARAMETER_* schemas are no references
anymore to allow using them as external schemas via the
`"schema"` key inside object schemas inside the `#[api]`
macro.

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

completion: COMP_POINT is a codepoint count not a byte index

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

compile fixups

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

add websocket (default-)feature, make futures optional

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

make the proxmox crate without default features again

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

some minor clippy lint cleanups

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

ByteBuffer: use T: AsyncRead, not AsyncReadExt

it's just weird otherwise

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

test cleanups

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

ByteBuffer: don't require Sized on Read references

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

ByteBuffer: actually test the doc comment code

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

ByteBuffer: skip the temporary Vec

The docs say `into_boxed_slice()` "drops" excess capacity,
but doesn't specify whether that just means it becomes
inaccessible or the vector potentially reallocates to the
exact size.

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

bump version to 0.1.42-1

sys/linux/pty: add examples and docs

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

tools/byte_buffer: add examples and docs

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

tools/websocket: add examples and docs

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

tools/websocket: improve traits for WebSocketReader

by dropping unecessary trais for R
and impl Sync for ReaderState when R implements Sync

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

tools/websocket: require writer to be Unpin

so we can safely pin/unpin it

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

tools/websocket: make mask not a reference

so we can drop the dereferencing, also use shorter syntax for mask_bit

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

tools: add Websocket helper

for now there is a helper for Reading and Writing
still missing is a utility for the handshake/accepting a connection

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

sys/linux: add PTY helper

for creating a PTY and make the secondary part a
controlling terminal for a child process

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

tools: add ByteBuffer

this is a Buffer struct that holds u8 and has convenience methods
for reading onto the back and consuming from the front

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

cleanup: libc has 'getrandom' since 0.2.59

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

bump proxmox to 0.1.41-1

api/router: percent decode the components of the uri

those can contain percent encoded values, so we have to decode them
before lookin up the path

this is especially important for the 'MatchAll' branch
when the parameter can contain some reserved characters

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump proxmox to 0.1.40-1

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

tools: add mmap helper

Mmap<T> acts like a slice of `T`s, inhereting its `Send` and
`Sync` traits

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

AsHex Display helper

This way we can use
    println!("something: {}", AsHex(bytes));

without having to turn them into an allocated string first.

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

add TtyOutpu helper and use it for read_password

This way read_password() will not write the prompt into
stdout if stdout was redirected into a file, but instead
opens `/dev/tty`.

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

bump version to 0.1.39-1

fix Permission::Privilege path check

when checking a path, we only pushed the components to the checked value
that would contain a parameter and not fixed ones

e.g. from the path components (with params datastore="foo")
["datastore", "{datastore}"]

only ["foo"] was given to the lookup_privs function
instead of ["datastore", "foo"]

this fixes an issue where e.g. a prune on /datastore/foo would fail
for a user that has the role DatastorePowerUser on /datastore (with
propagate) or /datastore/foo

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

add tests for check_api_permission

add tests for check_api_permission and many Permission combinations
this test fails for now and will be fixed with the next commit

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 0.1.38-1

proxmox/src/tools/fs.rs: add additional helpers to get file contents

bump version toö 0.1.37-1

proxmox: add string_as_base64 serde (de)serializer

this can be used to automatically convert string fields to base64

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump proxmox to 0.1.36-1

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

procfs: iowait_percent fixup

Fix time delta calculation, and use old percentage when
querying more often than once a second.

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

formatting fixup

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

bump proxmox to 0.1.35-1

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

procfs: add loadavg reader

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

make CLOCK_TICKS public

The ProcFsStat type uses values which need to be scaled by
it, so it makes sense to have this be public.

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

procfs: add iowait_percent

The calculation for this is based on the pve code.

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

procfs: don't count guest and guest_nice times twice

See the following kernel code snippet from
kernel/sched/cputime.c's account_guest_time():

    /* Add guest time to cpustat. */
    if (task_nice(p) > 0) {
        cpustat[CPUTIME_NICE] += cputime;
        cpustat[CPUTIME_GUEST_NICE] += cputime;
    } else {
        cpustat[CPUTIME_USER] += cputime;
        cpustat[CPUTIME_GUEST] += cputime;
    }

So the old sum was wrong as some time frames were counted
double in the `total` value.

The data will likely still deviate from PVE, but we should
check by how much, as this version does seem more accurate.
We can still fix it to just use the same math as pve if it
seems off.

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

bump version to 0.1.34-1

add 'constnamemap' macro

to be able to define constant values but still have some mapping to
the name of it

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

pumb version to 0.1.33-1

proxmox/src/api/section_config.rs: simplify code

section_config: fixup fallback schema in parse

we also have to fallback to the id_schema of the section_config when
we parse it, not only on write

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 0.1.32

proxmox: section_config: add convert_to_typed_array

converts a parsed section config to an Vector of typed items

this way we can simply get a list of users from the user config

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

proxmox: section_config: add id_property to SectionConfigPlugin

if set in the SectionConfigPlugin, we set the given
field to the section_id for each entry

this way we can have e.g. the userid in the Struct and ObjectSchema,
without having to write it twice to the config but we are still getting
it when parsing the config

when a plugin does not specify an id_property, then the id_schema of
the SectionConfig will be used as fallback

also adds a test for this with multiple plugins

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 0.1.31-1

proxmox/src/api/rpc_environment.rs: use Value to store metadata

And impl. Index and IndexMut to make it easy to access/set the metadata.

update dependencies

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

api-macro: bump version to 0.1.9-1

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

bump version to 0.1.30-1

proxmox/src/tools/time.rs: binding for libc localtime/timelocal

api-macro: remove more unnecessary parentheses

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

api-macro: remove some unnecessary parentheses in output

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

bump version to 0.1.29-1

proxmox/src/api/section_config.rs: make record_order public

It is sometimes usable to record the order.

bump version to 0.1.28-1

proxmox/src/api/section_config.rs: add support for systemd config files

proxmox/src/api/section_config.rs: support array properties

bump version to 0.1.27-1

schema: split property string lists with comma, semicolon and white space.

bump version to 0.1.26-1

run_cli_command: add additional rpcenv parameter

bump version to 0.1.25-1

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

bump api macro to 0.1.8

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

implement descriptions for enum variants

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

fix changelog entry for proxmox 0.1.23

bump proxmox version to 0.1.24

proxmox/src/api/section_config.rs - convert_to_array: remove static lifetime

not required...

src/api/cli/text_table.rs - format_object: skip optional properties without data

bump proxmox version to 0.1.23

api/cli/command.rs: handle fixed_param

Revert "proxmox: add SysError::is_errno_raw"

Not actually required, the numbers just overlapped and are
therefore no separate enum entry.

This reverts commit 8f57f29ca84dbfdb99a822fdfbba43b361577a81.

proxmox: add SysError::is_errno_raw

to work around missing errno values in nix 0.16

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

fixup last commit

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

proxmox: tools: add RawFdNum helper

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

api/cli/completion.rs: implement completion for lists of strings

IP_REGEX - match whole string

bump proxmox crate to 0.1.22-1

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