vzdump: use strftime for backup basename assembly

we already use POSIX strftime a lot in the stack, so nothing new,
also just use the perl built in localtime

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

fix #263: only include node rrd stats if user has Sys.Audit

It makes sense to not give users without Sys.Audit permissions to
much information over a node and this is relatively easy and cheap to
check and enforce at those two points.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Tested-by: Oguz Bektas <o.bektas@proxmox.com>

followup: code cleanup and print error

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

pveceph: install ceph service print sucess message

This patch adds a success message on successful ceph.service
installation. And adds a newline to make a successful ceph package
installation more visible.

Signed-off-by: Alwin Antreich <a.antreich@proxmox.com>

fix #2177: use getValue() to get otp value

as '.value' isn't always valid

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

tfa: disable, not mask, other cards if tfa is already setup

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

tfa: comment that TOTP secret is meant to be base32

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

make jslint happier

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

ui: tfa: set initial tab based on current tfa type

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

ui: tfa: mask panels for not currently used types

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

ui: user view: show tfa type name

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

www: parser: add parseTfaType helper

The tfa keys can now contain the type as additional
information, so let's parse it out.

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

d/control: bump widget-toolkit version dependency

It's not that nothing would work without it, but ceph cannot be
configured via gui, and knowing that some user want to do strange
things with mixing package versions, we save us some hassle just
recording this to ensure one cannot run into this..

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

fix ceph install wizard not accepting ipv6

changes the vtype to the newly defined IP64CIDRAddress. depends on the
introduction of the IP64CIDRAddress vtype in proxmox-widget-toolkit.

Signed-off-by: Mira Limbeck <m.limbeck@proxmox.com>

d/control: bump doc-generator build dependency version

When building a current version of pve-manager a dependency to the
documentation is missing, as it ships new references used by our
onlineHelp tooling this is required.

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

cleanup Status plugins use statements

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

followup: code cleanup, remove unnecessary type check

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

fix #1326: allow multiple status server definitions per type

we allow an id like storage.cfg but leave it optional (so we do not
break existing configs):

 influxdb: name

so that one can export the data to multiple servers of the same type

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

bump version to 5.4-3

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

ui: dc/options: allow to edit cluster wide migration settings

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

followuop: ui/dc/options: fix default

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

ui: dc/options: allow to edit HA shutdown policy

add some helpers handling this, can be re used for adding the
migration field, maybe we want to move this to widget-toolkits parent
ObjectGrid class, a bit cleaned up.

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

ui: lxc: do not disable complete add resource button if no disk caps

While for now you only have elements which require the add disk
capability in the add sections it makes no sense to hide it, as it
would suggest that one can add no resource at all, and can easily
introduce a bug if we add another type of resource here, independent
of disk add cap.

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

fix #1814: do not enable disk related buttons for audit user

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

ui: u2f: refactor error code map out and use in login window

to avoid showing numbers as error codes to users, even though the
strings are not much more helpful either...

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

remove non-existant property

vmComboSelector does not have such a property

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

ui: Restore: Fix error where the textfield did not default to next free VMID

This was introduced when refactoring our seprated CT & VM ID
selectors into an unified one in
commit ed416f20248d46d95c9d1547a2e1b833aeb097d6

Signed-off-by: Christian Ebner <c.ebner@proxmox.com>

use correct icon for hibernate in cmdmenu

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

fix jslint warning

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

bump version to 5.4-2

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

d/control: bump version dependency to libpve-access-control

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

api: ceph/mon: improve no ip error

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

ui: tfa: add onlineHelp

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

ui: qemu system panel: set onlineHelp

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

Ceph MON - Reword error message on IP config

Reword the error message in find_mon_ip to make it more clear, that
there is no active IP configuration for the ceph public network.

Signed-off-by: Alwin Antreich <a.antreich@proxmox.com>

ui: tfa login: wording followup

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

ui: login: remove fixed OTP field completely

we now have the TFALoginWindow which pops up if TFA is configured,
so use only this single stream lined approach.

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

ui: use 2-step authentication for non-tfa realms

(Ext.Msg.prompt() fails on firefox...)

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

HTTPServer: verify_ticket now returns a hash

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

Revert "ui: always show otp field"

This reverts commit 1702b529b66307f9aeafc0e9f28c7afe3864ffbb.

ui: ceph install: open ceph.com in new window

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

pvereport: only comput report def if we generate a report.

it's really not a good idea to start doing exec, file exist checks,
... on every module load of this, this may even hang completely
differen parts if something pull us in, and confuse one completely...

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reported-by: Wolfgang Link <w.link@proxmox.com>

followup: reduce wording, less text has better chance to be read

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

cleanup format.

Supplementary Ceph installation summary

Add installation node for the other participant Ceph Node.

update shipped aplinfo index

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

ui: improve datacenter summary ceph node selection

instead of trying a single random node 3 times, try all nodes
until we either find one with ceph, or until we tried them all

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

ui: make Datacenter subscription info clickable when not valid

to make it easier for the user to buy the subscriptions

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

ui: fix blinking subscription info panel

not having a subscription trumps having mixed subscriptions, so we
check that correctly

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

ui: tfa: also include validity check for password field

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

ui: tfa: use userid directly for totp url generation

as name is now a displayfield, which by default does not
gets submitted, so just use the fixed userid directly

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

ui: tfa: show username as read only display field for info

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

ui: tfa: remove fixed labelwidth

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

ui: tfa: allow apply only on valid inputs

only when the totp form and the challenge is valid, allow pressing the
apply button, default is disabled, as the 'user_tfa' data binding was
not used anywhere else replace it with something more fitting.

change allowBlank for the challenge

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

ui: qemu: change suspend to disk icon to fa-download

it ressembles a state downloaded to disk, so quite fitting for
hibernate/suspend to disk

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

add a filter mask for the TFA Secret

this way a user cannot (easily) enter wrong characters
else if an invalid Character is entered, one can still hit apply
but not all characters will be used for the secret

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

ui: parser use parseBoolean for firewall

so we detect on/ff,true/false,1/0

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

bump version to 5.4-1

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

ui: ct: fix displaying nic firewall if explicitly 0

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

ui: restore: add tooltip for unique checkbox and declare static

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

fix: #1218 Expose unique MAC address generation on restore of VMs and CTs in UI

Adds a checkbox to the restore window, allowing the user to avoid MAC address
collisions when restoring a VM/CT to a different vmid.
When restoring to the same vmid the checkbox is not visible.

Signed-off-by: Christian Ebner <c.ebner@proxmox.com>

ui: followup trailing comma fix

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

ui: tfa: fix secret validator

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

ui: user view: improxe "has TFA" check

detect also our old variant for TFA, so all users where the keys
values is non-empty have TFA

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

ui: user edit: move TFA key IDs field to advanced section

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

LoginWindow: make static

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

ui: tfa: move qrcode creation to controller

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

ui: tfa: make lint happy, declare globals

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

ui: tfa: also do not allow to delete TFA if not set

better UX

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

ui: tfa: pregenerate secret if user has not setup TFA yet

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

ui: userview: show TFA column

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

ui: tfa: hide digit/duration fields as some auth apps are broken

Google Authenticator, for example, just ignores it and thus one
cannot use they produced verification codes...

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

ui: tfa: follow up cleanups

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

ui: tfa: s/Code/Verification Code/

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

ui: tfa: ExtJS debug complains about - in references

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

ui: tfa: make jsliunt a bit more happy

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

ui: tfa: move randomize button right of secret field

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

ui: include qrcode and u2f libs also if debug is on

because it's really not ideal if enabling debug breaks things...

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

ui: TFA: hide accout name

It's best to let this to the users real account name, it just
clutters the interface, an there's no real value in it.

One can already use the "issuer" field to add personalized info.

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

d/control: bump version dependency to proxmox-widget-toolkit

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

d/control: bump version dependency to libpve-access-control

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

ui: support u2f authentication and configuration

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

ui: always show otp field

But make it optional.

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

ui: add qrcode.min.js

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

add u2f-api.js

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

store u2f challenges in the rpc environment

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

fixup whitespaces

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

Add Ceph, iscsi and more disk and zfs information to pvereport

Following items added to pvereport:

* more details about disks
* iscsi information
* more zfs information
* ceph information (only when ceph.conf exists)

When running from command line now report progress is shown at the console (STDERR)

fix: #1145 enable CT/VM firewall by default on creation via WebUI

Signed-off-by: Christian Ebner <c.ebner@proxmox.com>

follow up: newline at end of file

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

adapt ceph log for new logview

since there is no 'doAttemptLoad' anymore, we use the
failCallback to overwrite the behaviour

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

Allow to set the firewall log rate limit and burst from the UI

Signed-off-by: Christian Ebner <c.ebner@proxmox.com>

ui: ceph installer: set initial tab depending on current state

Signed-off-by: Tim Marx <t.marx@proxmox.com>

ui: ct/vm: add tooltip to hibernate and stop button

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

gui: show Resume button if suspended

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

gui: add Hibernate/Pause to CmdMenu/Shutdown Menu

and have it in order of impact on the guest
(pause, hibernate, poweroff, stop)

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

gui: CmdMenu: rename suspend to paus

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

gui: CmdMenu: hide the start and suspend button conditionally

only to disable them wastes space for no reason and might confuse users

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

fixup trailing whitespaces

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

ui: ceph install: remove advanced option pg_bits

Signed-off-by: Tim Marx <t.marx@proxmox.com>