ui: tfa: make webAuthn abortable and restartable

Fix two things:
* do not reject the login promise when we get the abort DOMException
  error
* safely save the original challenge string as we work on a reference
  here and avoid to convert to a UInt8 array twice to avoid an
  exception.

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

ui: tfa: only immediately trigger webAuthn when its the initial tab

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

ui: tfa: drop bogus console.error

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

ui: tfa: save last used TFA method and prefer it next time

simple heuristic for those people who always prefer a specific TFA
method and have the others only as backup.

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

fix build: commit missing file

ui: rework TFA prompt on login

Improve UX by avoiding the need to click some buttons twice, or
calling TOTP and Recovery codes both "OTP" codes and showing multiple
buttons, with all having the same goal "submit a TFA token" at the
same time.

Instead use a tab panel with a single submit button.

WebAuthn can and should be still improved, but that can be OK as
followup.

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

pmtx: implement scan command

remove generated file synopsis.rst (no need to track in git)

tape: use 36 byte Inquiry (recommended size)

install pmtx binary

cleanup: move scan changers API implementation

tape: rename retry_command to execute_scsi_command, make retry a flag

tape: repeat changer scsi command until successful

sgutils2: use sg_get_asc_ascq_str to produce error messages

sgutils2: add ASC codes from tandeberg docs

sgutils2: further improve error messages

sgutils2: support RequestSense Descriptor format

tape/changer/mtx: add mtx parser test

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

api2/tape/changer: reorganize api

add a changer listing here (copied from api2/config/changer)
and put the status and transfer api calls below that

puts the changer scan into the top level tape api
and removes the (now redundant) info from the config api path

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

tape: fix typos

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

docs/tape: fix some typos and improve wording

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

sgutils2: improve error messages

tape: add new command line tool "pmtx"

Also improve sgutil2 error reporting

ui: add missing uri encoding in user edit and view

userid parameter needs to be properly encoded when shown on the browser

Signed-off-by: Oguz Bektas <o.bektas@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

server/worker_task: improve newline handling in upid_read_status

improves upid_read_status with:
* ignore multiple newlines at the end
* remove all code that could panic (array index access)
  the one place where we access with '[pos+1..]' is ok since
  we explicitely test the len of the vector, this is done to
  let rust optimize away the range checks, so it cannot panic

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

pxar: don't clone patterns unnecessarily

The options struct has no Drop handler and is passed by-move
so we can partially move out of it.

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

clippy: more misc fixes

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

clippy: allow api functions with many arguments

some of those can be reduced/cleaned up when we have updater support in
the api macro.

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

verify: factor out common parameters

all the verify methods pass along the following:
- task worker
- datastore
- corrupt and verified chunks

might as well pull that out into a common type, with the added bonus of
now having a single point for construction instead of copying the
default capacaties in three different modules..

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

derive/impl and use Default for some structs

and revamp HttpClientOptions with two constructors for the common use
cases

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

authid: make Tokenname(Ref) derive Eq

it's needed to derive Hash, and we always compare Authids or their
Userid components, never just the Tokenname part anyway..

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

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>