ui: pbs remote: use proxmoxtextfield for password edit xtype again

to get back the skipEmptyText option

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

object editors: rework so that default label width fits everywhere

mostly by s/Regular Expression/Regex/ and transforming some
checkboxes' fieldLabel into boxLabels

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

regex tester: make textfield flex to take all remaining space

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

pbs remote: add namespace support

restructure the existing field positions a bit to make it more
balanced with the new NS field.

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

pbs remote: disallow editing fixed fields

similar in spirit of PVE's storage entries for PBS, changing those
after create can make backups disappear and just shouldn't be done,
user should add a new remote instead.

Same for user name, changing it will make backup almost definitively
fail the owner check, so only keep the password field, for PW
rotations (we should disable it for tokens though).

For UX it's better to check if params are correct and the repo is
accessible on create and to add scan datastore and namespace
endpoints to allow a friendlier selection of those, avoiding typos.

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

utils: code cleanup

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

bump version to 3.1-2

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

sorters: use correct property 'direction' and keep default 'ASC'

Ext.util.Sorter does not have an 'order' property, so 'order: DESC'
didn't have an effect. The default is 'ASC' and it is arguably the
preferred direction for all affected sorters anyways.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

mail header view: code reduction

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

mail header view: switch toggle-raw button to check box and move into panel

Allows an user to actually see what the current state is and wastes
less space for the actual header view.

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

postfix queues: show decoded headers by default

and add a toggle button to show the raw headers

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

mail info: move receiver to top right

tying to meake better use of empty space

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

rename recipient to receiver

we always call it receiver, not recipient, saves us a translation string

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

quarantine: mail display: show receiver information

Showing the sender and subject is sufficient in the case of the
spam-quarantine - when viewed by a single user.

If an admin views any of the quarantines the information of the
recipient should be helpful in general, but is needed in case a mail
was sent to multiple recipients.

Reported in our community forum:
https://forum.proxmox.com/threads/.103276/

Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>

bump version to 3.1-1

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

tfa: add webauthn config panel

keep it simple and avoid an extra options panel

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

tfa: set issuer name to something more telling

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

tfa: move tab-panel besides User one

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

d/control: add libjs-qrcodejs dependency

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

add TFA components

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

fix #3648: ldap config: do not send empty bindpw

so that we can reuse the saved one and the admin does not have
to enter the password on each config change (similar to PVE).

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

bump version to 3.0-5

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

fix #3565: fix typo in gettext

s/orignal/original/

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

d/control: bump versioned dependency for proxmox-widget-toolkit

for factored out node repo-status widget

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

dashboard: node status: use repo status widget from widget-toolkit

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

dashboard: node info: use local subscription status only

instead of the cluster-wide status.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

utils: align notification toast to bottom-right again

The default changed sometimes between ExtJS 6.0.1, which used 'br',
and ExtJS 7.0, which has 't' (top) as new default.

So set it explicitly to the previous used one, less intrusive and was
intended to be there.

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

bump version to 3.0-4

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

spam quarantine: avoid resetting saved position when removing single element

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

server status: add button to show package version window

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

server administration: repos: add online help

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>
Reviewed-by: Stoiko Ivanov <s.ivanov@proxmox.com>
Tested-by: Stoiko Ivanov <s.ivanov@proxmox.com>

sys config: network: show live-apply button

The PMG API supports that feature now when ifupdown2 is installed.

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

sys config: move network to bottom

Like we did in PBS, which has a very similar GUI structure as PMG.

Rationale is that the network panel is flexed and thus normally the
biggest, while DNS and time are fixed and short, and an inverted
pyramid, height-wise, looks a bit off as layout.

Also network will show the pending changes at the bottom, which is
nicer if its really at the bottom of the view.

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

bump version to 3.0-3

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

comment text width improvements

use the full 8ß to 100 CCs available

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

configuration: add icons

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

user management: add icons

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

server certificates: add icons

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

server administration: add icons

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

dashboard: increase height of second row slightly

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

dashboard: add hover text-shadow class for repo-panel link

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

css: add .pmx-icon-size

from PBS, we probably should just add it to widget-toolkits css

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

dashboard: move cluster resources to end to avoid "jumping"

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

node info: avoid setting undefined in view model

undefined is used as delete when set to a normal data-path of a view
model, so, we need an actual defined value here as else we cannot
decide if the view model was ever initialized or not.

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

dashboard: hide cluster resources if it's a single node

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

dashboard: add node info panel

Essentially the same as in PBS, but needed to use 'rootfs' instead of
'root' for the disk usage, avoid 'cpu' as an itemId (already used by
the cluster resources), and get rid of the fingerprint button. Also
added the link to the repositories panel like in Dominik's pending
patch for PBS.

Rename the existing resources panel to clarify what it actually is.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

d/control: bump versioned dependency for proxmox-widget-toolkit

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

add panel for APT repositories

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

add Settings window

copied from pbs and changed PBS to PMG

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

ServerStatus: update column width on state change

when having the settings window (when implemented) open there

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

LoginView: add save username checkbox

like we have in pbs/pve but only for non-quarantine view

we have to increase the width of the window, so it fits together
with the 'send quarantine link' button

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

server status: refactor and code cleanups

arrow functions all the way ;-)

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

status: avoid using depreacated helper

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

ServerStatus: update panel columsn dynamically

like we do in pve and pbs
for now the user cannot change the preferred columns and the default is 'auto'

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

ui: minimally increase font-size of product title and version

Similar like we did for Proxmox VE's manager. The main title and
version should stand a bit more out compared to simple nav/button
texts.

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

bump version to 3.0-2

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

quranatine view: make navtree more like main navtree

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

quarantine view: change ui to pve-nav

since 'nav' contains some styling we do not want (in extjs 7.0)

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

main view: fix nav tree indentation

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

quranatine view: adapt to new beforeChangePath signature to fix routing/app

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

buildsys: upload to bullseye

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

bump version to 3.0-1

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

virusquarantine: fix time format

Ext js orients itself on php's date function, which uses 'i' for
minutes ('m' is the month as integer)[0].

Reported via our community-forum:
https://forum.proxmox.com/threads/time-display-bug-in-virus-quarantine.90816/

Quickly tested on my test-setup

[0] https://docs.sencha.com/extjs/7.0.0/modern/Ext.Date.html
Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>

d/control: bump compat level to 12

Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>

remove safe browsing from clamav tab

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

MainView: fix redirectTo call

takes now an object as paramter in extjs 7.0

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

MainView: use new beforeChangePath signature

subpath can be optional in extjs 7.0, so handle that

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

{Attachment, Spam, Virus}Quarantine: fix layout for iframe

in extjs 7.0 the css changed slightly so that we need to define
the 'fit' layout explicitely to maintain the full width/height

otherwise it can happen that the iframe gets cut off

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

change ui to 'pve-nav'

since 'nav' contains some styling we do not want (in extjs 7.0)

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

bump version to 2.4-3

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

quarantineview: unify url generation

this patch adds the cleanup for the attachment quarantine from
0e26e20aa5522b3f7d05e68c96104b051419b901 to the virus- and
spamquarantine.

Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>

attachmentquarantine: fix missing '&' for raw-param addition

small glitch introduced in the code cleanup in:
0e26e20aa5522b3f7d05e68c96104b051419b901

Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>

bump version to 2.4-2

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

quarantine: restore behavior of selecting next in list after action

This fixes a UX regression stemming from the recently introduced
commit 9aed379a700a0bc3bc26fa32d43a5bab75eb206b
"spam quarantine: avoid full store reload on single-element actions"

A load restored the saved selection after load complete, with the
neat and very useful feature to jump to the next in line if the old
one did not exist anymore.

Call this now factored out code part ourself in the fast-path.

This was reported in the forum:
https://forum.proxmox.com/threads/proxmox-mail-gateway-6-4-released.86760/#post-380971

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

quarantine list: factor out restoreSavedSelection

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

code cleanup

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

spam quarantine: code cleanup

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

spam context menu: set correct handler for whitelist/blacklist

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

bump version to 2.4-1

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

d/control: update versioned dependency for widget-toolkit

ensure that the new Proxmox.Async class is available and that we have
all of the checkbox selection model performance tuning available.

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

spam quarantine: avoid full store reload on single-element actions

A store load may require several seconds to complete, which can get
frustrating fast.

For single actions we have a callback which gets only executed on a
successful API call, so we can just remove the entry there directly
from the in-memory store.

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

spam quarantine: batch actions into 2500-sized chunks

With ~20 bytes per message-id and separator we get over the current
HTTP post size limit of 64 KiB with ~ 3200 messages.

As 2500 is a relatively nice number for humans to deal with rather go
for that one, that leaves also some room for potential longer IDs in
the future.

Send the batches of async in pairs, as with that the pmgproxy can
still cope well and is not overloaded.

Mask the whole list panel with a 'Processing...' loading-mask during
sending of requests affecting multiple messages, so that the user
gets some instant visible feedback that something is actually
happening (the api calls may return only after a few seconds).

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

spam quarantine: shorter mapping from selected to id list

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

fix #3164: allow one to display all quarantined spam mails

If the API call returned more than one pmail entry, inject an "all"
entry which, if selected, drops the user parameter and loads the
quarantine mails of all users from the backend.

The webinterface has only some issues regarding deselection (all in
the grid header or if we need to deselect due to the search filtering
out some elements) - for that the underlying issue was found and a
widget toolkit patch was provided.

The rest seems now pretty performant, albeit more than a few 100k
mails may become a problem here. But, in such big setups the mail
admin won't tinker to much whith the users mail anyway, if they are
even alowed to do so depending on their jurisdictions privacy laws
and companies privacy policy.

So, basically this is more for evaluation or for smaller setups but
got quite often requested, and as there's not more data
exposed/returned then already available I see no real argument
against it.

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

quarantine list: factor out actual filtering in separate function

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

qlist: code cleanup

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

quarantine list: allow one to filter by subject or from sender

Deselect those entries which got filtered out, to avoid doing
some action (delete, deliver) on hidden records.

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

quarantine list: code cleanup

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

bump version to 2.3-3

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

certs: show onlineHelp tool linking directly to context chapter

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

acme config view: derive from panel not container

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

utils: add onlineHelpTool

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

add certificates and acme view

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

bump version to 2.3-2

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

buildsys: enforce strict eslint (no warnings allowed) in check target

the for development designed incremental lint target is still non
strict, and allows warnings allowing less interrupted dev workflow

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

pbs remote: notify: use gettext and fix casing

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

d/control: update versioned dependency to pmg-doc-generator for new anchors

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

backup: pbs: fix #3146 add notify setting to remote

Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>

backup: pbs: fix #3154: add statistic setting to remote

Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>

backup: fix #3154: make statistic backup optional

by creating an edit-window when clicking on the respective Backup now
button.

This is the second part of the enhancement request

Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>