pxar: extract PxarExtractOptions

same as PxarCreateOptions, but for extraction/restore rather than
create.

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

pxar: factor out PxarCreateOptions

containing the CLI parameters that are mostly passed-through from the
client to our pxar archive creation wrapper in pxar::create

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

pxar: typedef on_error as ErrorHandler

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

client: factor out UploadOptions

to reduce function signature complexity.

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

systemd/time: extract Time/DateSpec structs

could be pulled up into CalendarEvent if desired..

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

async index reader: typedef ReadFuture

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

allow complex Futures in tower_service impl

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

client: refactor catalog upload spawning

by pulling out Result type into separate struct

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

broadcast_future: refactor broadcast/future binding

into its own, private struct.

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

report: type-alias function call tuple

to make clippy happy.

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

tools::sgutils2: name fixup

it's not a box anymore

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

tools::sgutils2: use NonNull

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

tools::sgutils2: extern 'C' and import ordering

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

tools::sgutils2: don't transmute to a Box

Otherwise we run the drop handler for the scsi pt object AND
the box itself, which shouldn't even work as it should be
doing a double-free (unless the library does some kind of
reference counting in which case this should simply crash
later on?)

anyway, let's make a wrapper simply called `SgPt` containing
the pointer from `construct_scsi_pt_obj()`

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

tools::sgutils2: const correctness

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

tape: cleanup - use ScsiMediaChange trait instead of mtx_status()

tape: add/use rust scsi changer implementation using libsgutil2

sgutils2: add scsi_inquiry command

http-client: further clippy cleanups

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

http-client: fix typoed ticket cache condition

which was even copy-pasted once without noticing.

found with clippy.

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

rework GC traversal error handling

the error message don't make sense with an empty default

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

apt: let api handle optional bool with default

one less FIXME :)

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

clippy: rewrite ifs with identical return values

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

clippy: rewrite comparison chains

chunk_stream one can be collapsed, since split == split_to with at set
to buffer.len() anyway.

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

clippy: fix Mutex with unused value

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

clippy: add is_empty() when len() is implemented

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

sgutils: allow command which does not transfer any data

sgutils2: allow to set custom timeouts

cleanup: avoid compiler warnings

tape: add docu about paperkey

tape: document hardware encryption

tape: avoid error when clearing encryption key

Simply ignore clear request when sg_spin_data_encryption_caps fails.
Assume those are tapes without hardware encryption support.

docs: clarify that client-server communication is secure

This clarifies the fact that all communication between client and server
uses TLS for secure communication.

Signed-off-by: Dylan Whyte <d.whyte@proxmox.com>

improve code docs in api2

Note: API methos should be declared pub, so that they show up in the generated docu.

d/control: bump B-D on pve-eslint

the old one does not understand www/config/TfaView.js and fails the
build..

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

tape: add media pool config code docs

tape: add drive config code docs

fix mtx parser bug: s/strip_suffix/strip_prefix/

tape: implement paperkey command for tape encryption keys

paperkey: move code to src/tools/paperkey.rs

tape: add volume-statistics api/command

tape: add estimated medium wearout to status

tape: set encryption key on restore

tape: add code docs to src/config/tape_encryption_keys.rs

renamed src/tape/sgutils2.rs -> src/tools/sgutils2.rs

tape: move MediaCatalog magic number into struct (doc cleanup)

tape: further hierarchy improvements

tape: change code hierarchy to improve docs

tape: do not export/doc low level libsgutils2 bindings

ui: cleanup order of declraing properties

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

ui: fix on-parse use of global Proxmox.UserName

This is wrong most of the time, when not loading the web interface
with valid credentials, and thus some checks or defaults did not
evaluated correctly when the underlying value was only set later.

Needs to be set on component creation only, this can be done through
initComponent, even listeners, view controllers or cbind closures.

Use the latter, as all affected components already use cbind.

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

tape: improve code docs

cleanup: always compute fingerprint in KeyConfig constructors

Merge branch 'master' of ssh://proxdev.maurer-it.com/rust/proxmox-backup

tape: try to set encryption key with read-label command

tape: restore_key - always update key, even if there is already an entry

tape: fix file permissions for tape encryptiuon keys

cleanup: remove missleading wording from code docs

cleanup: KeyConfig::decrypt - show password hint on error

tape: use type Uuid instead of String

tape: use specialized encryption key per media-set

api: add types for UUIDs

clippy: replace transmute with &*

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

clippy: remove drop(&..)

it does nothing.

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

clippy: use is_null to check for null pointers

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

clippy: remove some unnecessary reference taking

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

clippy: fix for_kv_map

and allow it in the one case where the entry loop is intended, but the
code is not yet implemented fully.

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

clippy: fix/allow needless_range_loop

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

clippy: use while let loops

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

clippy: pass &str/&[..] instead of &String/&Vec

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

clippy: remove unnecessary &mut

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

clippy: remove unnecessary `let` binding

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

clippy: remove/replace needless explicit lifetimes

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

clippy: fix/allow identity_op

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

clippy: us *_or_else with function calls

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

clippy: convert single match to if let

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

clippy: misc. fixes

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

clippy: don't pass along unit value

make it explicit. this whole section should probably be re-written with
select!

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

clippy: use unwrap_or_default

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

clippy: use copied/cloned instead of map

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

clippy: shorten assignments

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

clippy: fix option_as_ref_deref

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

clippy: use matches!

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

clippy: collapse nested ifs

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

clippy: avoid useless format!

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

clippy: remove explicit returns

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

clippy: use chars / byte string literals

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

clippy: is_some/none/ok/err/empty

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

clippy: remove needless bool literals

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

clippy: use strip_prefix instead of manual stripping

it's less error-prone (off-by-one!)

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

clippy: drop redundant 'static lifetime

those declarations are already const/static..

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

clippy: remove unnecessary closures

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

clippy: remove unnecessary clones

and from::<T>(T)

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

clippy: collapse/rework nested ifs

no semantic changes (intended).

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

tape: allocate new media set when pool encryption key changes

improve code docs

tape: implemenmt show key

Moved API types Kdf and KeyInfo to src/api2/types/mod.rs.

tape: implement change-passphrase for tape encryption keys

tape: add --kdf parameter to create key api

cleanup KeyConfig