tfa: remove/empty description for recovery keys

While the user chosen description is not allowed to be
empty, we do leave it empty for recovery keys, as a "dummy
description" makes little sense...

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

ui: tfa: fix ctime column width

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

gui: tfa: make description fill the remaining space

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

gui: tfa: show when entries were created

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

tfa: add 'created' timestamp to entries

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

docs: fix typo in client manpage

Signed-off-by: Oguz Bektas <o.bektas@proxmox.com>

gui: enumerate recovery keys and list in 2nd factor window

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

tfa: remember recovery indices

and tell the client which keys are still available rather
than just yes/no/low

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

tape: implement encrypted backup - simple version

This is just a proof of concept, only storing the encryption key fingerprint
inside the media-set label.

implement FromStr for Fingerprint

tape: add encrypt property to media pool configuration

tape: clear encryption key before writing labels

We always write labels unencrypted.

tape: add hardware encryption key managenent api

ui: tfa: fix title for removal confirmation

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

ui: tfa/recovery: add print button to key info window

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

ui: tfa/recovery: fix copy button text, add icon

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

ui: tfa/recovery: disallow to close key info window with ESC

to avoid accidental closing it

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

ui: tfa: specify which confirmation password is required

Clarify that the password of the user one wants to add TFA too is
required, which is not necessarily the one of the current logged in
user. Use an empty text for that.

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

ui: tfa totp: whitespace and padding fix

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

ui: access: stream line add/edit/.. button order and separators

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

pull: add error context for initial group list call

otherwise the user is confronted with a generic error like "permission
check failed" with no indication that it refers to a request made to the
remote PBS instance..

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

pull: rustfmt

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

tape: encryption scsi command cleanup

tape: add scsi commands to control drive hardware encryption

tape: sgutils2.rs - add do_out_command()

Make it possible to run commands that writes data.

ui: webauthn config: set default values for unconfigured case

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

ui: webauthn config: use ID instead of Id/id

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

gui: tfa configuration

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

tfa: add webauthn configuration API entry points

Currently there's not yet a node config and the WA config is
somewhat "tightly coupled" to the user entries in that
changing it can lock them all out, so for now I opted for
fewer reorganization and just use a digest of the
canonicalized config here, and keep it all in the tfa.json
file.

Experimentally using the flatten feature on the methods with
an`Updater` struct similar to what the api macro is supposed
to be able to derive on its own in the future.

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

bakckup::manifest: use tools::json for canonical representation

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

add tools::json for canonical json generation

moving this from backup::manifest, no functional changes

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

tape: do not abort backup if tape drive does not support tape-alert-flags

tape: update restore docu

d/rules: patch out wrongly linked libraries from ELFs

this is a HACK!

It seems that due to lots of binaries getting compiled from a single
crate the compiler is confused when linking in dependencies to each
binaries ELF.

It picks up the combined set (union) of all dependencies and sets
those to every ELF. This results in the client, for example, linking
to libapt-pkg or libsystemd even if none of that symbols are used..

This could be possibly fixed by restructuring the source tree into
sub crates/workspaces or what not, not really tested and *lots* of
work.

So as stop gap measure use `ldd -u` to find out unused linkage and
remove them using `patchelf`.

While this works well, and seems to not interfere with any debug
symbol usage or other usage in general it still is a hack and should
be dropped once the restructuring of the source tree has shown to
bring similar effects.

This allows for much easier re-use of the generated client .deb
package on other Debian derivaties (e.g., Ubuntu) which got blocked
until now due to wrong libt-apt verison or the like.

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

debian/control: set VCS urls

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

d/control: sort and fix whitespace errors

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

access: restrict password changes on @pam realm to superuser

for behavior consistency with `update_user`

Signed-off-by: Oguz Bektas <o.bektas@proxmox.com>

access: limit editing pam credentials to superuser

modifying @pam users credentials should be only possible for root@pam,
otherwise it can have unintended consequences.

also enforce the same limit on user creation (except self_service check,
since it makes no sense during user creation)

Signed-off-by: Oguz Bektas <o.bektas@proxmox.com>

proxmox 0.10: adapt to moved ParameterSchema

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

cleanup: remove unnecessary 'mut' and '.clone()'

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

examples: unify h2 examples

update them to the new tokio-openssl API and remove socket buffer size
setting - it was removed from the TcpStream API, and is now only
available via TcpSocket (which can in turn be converted to a
TcpListener), but this is not needed for this example.

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

hyper: use new hyper::upgrade

the old Body::on_upgrade method is no more

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

tokio 1.0: update to new Signal interface

Signal does not yet re-implement Stream (and is not yet wrapped in
tokio-stream either).

see https://github.com/tokio-rs/tokio/pull/3383

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

tokio 1.0: update to new tokio-openssl interface

connect/accept are now happening on pinned SslStreams

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

tokio 1.0: use ReceiverStream from tokio-stream

to wrap a Receiver in a Stream. this will likely move back into tokio
proper once we have a std Stream..

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

tokio: adapt to 1.0 process:Child changes

Child itself is no longer a Future, but it has a new wait() async fn
that does the same thing

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

tokio: adapt to 1.0 runtime changes

enter() now returns a guard, and the builder got revamped to make the
choice between MT and current thread explicit.

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

tokio 1.0: AsyncRead/Seek with ReadBuf

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

proxmox 0.10: use tokio::time::timeout directly

TimeoutFutureExt is no more

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

tokio 1.0: delay -> sleep

almost the same thing, new name(s), no longer Unpin

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

update to tokio 1.0

and various related crates

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

d/control: sort and fix whitespace errors

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

manager: user/token list: fix rendering 0 (never) expire date

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

tape: sg-tape-cmd - add more ways to specify devices

tape: add --inventorize flag to read-label API/CLI

ui: window/{AddWebauthn, TfaEdit}: fix spacing/border of the windows

the password field should not be indented differently than the rest of
the fields, and we never have a border on the panels

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

ui: window/AddTotp: fix spacing styling of form fields

by moving the lower fields into the form itself and dropping the padding

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

ui: window/AddTfaRecovery: fix style of TfaRecoveryShow window

to have a more similar layout/spacing to our other windows

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

ui: window/AddTfaRecovery: rewrite to a Proxmox.window.Edit

we can reuse the edit window from widget toolkit for the most part
this solves some spacing and layout issues and is less code

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

ui: config/TfaView: disable Remove button by default

gets enabled when an item is clicked

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

ui: LoginView: remove not used viewModel

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

tape: MediaPool - allow to allocate free tapes

tape: rename changer_id to label_text

tape: docu - explain manual backups and tape cleaning

tape: improve docu

tape: MediaListEntry - add ctime

ui: tfa: drop bogus gettext of empty string

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

ui: notify options: Remove gettext for root@pam

Translating root@pam is not useful, especially as the empty text symbolises the
default value.

Signed-off-by: Dominic Jäger <d.jaeger@proxmox.com>

fix #3245: only use default schedule for new jobs

an empty schedule means 'none', so do not fill it with the default
in case we edit an existing job (like we do already for sync jobs)

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

tape: automatically reload tapes inside autoloader

We always automatically unload tapes to free library slots,
so it should not happen that an ejected tape resides inside the drive.

This is just a safe guard to handle the situation in case it happens ...

You can manually produce the situation by ejecting a tape without unloading:

 mt -f /dev/nst0 eject

Note: Our "proxmox-tape eject" does automatic unload

tape: make eject/export more reliable, improve logging

tape: improve backup logs

tape: fix tape alert flag values

d/control bump

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

api: return "invalid" as CSRF token for partial tickets

So that old clients don't `unwrap` a `None` value.

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

gui: masks for: adding recovery and removals

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

gui: add load mask during webauthn api calls

so that if we run into the 3s delay due to the wrong
password the window is properly masked

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

tfa: use UNAUTHORIZED http status in password check

to trigger our 3s delay in the rest handler

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

tfa: fixup for challenge file split

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

tfa: remove tfa user when a user is deleted

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

tfa: improve user existence check

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

tfa: allow deletion of entries of non-existent users

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

typo fixups

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

tfa api: return types and 'pub' structs/methods

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

tfa: entry access/iteration cleanup

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

tfa: r#type parameter name

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

tfa: _entry api method name suffix consistency

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

tfa gui: fix adding recovery keys as user

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

tfa view: html-escape description text

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

gui: tfa support

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

proxy: expose qrcodejs

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

depend on libjs-qrcodejs

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

api: tfa management and login

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

config: add tfa configuration

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

buildcfg: add rundir helper macro

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

tools: add create_run_dir helper

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

add tools::serde_filter submodule

can be used to perform filtering at parse time

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

tape: improve retention period docu

tape: remove unused eject_on_unload method

tape: more MediaChange cleanups

Try to provide generic implementation for complex operations:

- unload_to_free_slot
- load_media
- export media
- clean drive
- online_media_changer_ids