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>

backup: pbs: add onlineHelp anchors

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

d/control: slightly rework description

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

d/control: use https in homepage link

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

d/control: update dependencies, drop perl add eslint

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

statistics: use new api call for detailed stats

the new /statistics/detail api calls takes the type (contact, sender,
receiver) and address for which to display the statistics as explicit
parameter instead of path-component.

This makes it possible to accept '/' as part of an e-mail address
which is allowed (in the local-part by RFC5322 [0], and accepted by
postfix.

[0] https://tools.ietf.org/html/rfc5322

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

bump version to 2.3-1

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

add 'Request Quarantine Link' Button to LoginView

if the template has 'quarantinelink' enabled, we
show a button 'Request Quarantine Link' on the quarantine login ui

there a user can enter their e-mail and request a link to the quarantine

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

backup: rename manual backup button to "Backup Now"

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

pbs: improve schedule edit

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

pbs: add schedules grid, always select first remote

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

pbs: small cleanups and store field name fixes

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

pbs: factor out remote add/edit window and panels

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

pbs: factor out snapshot grid view

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

pbs: hide encryption column

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

pbs: followup: actually set cbind for all elements

we cannot have xtype nor cbind in the defaults, as moving xtype there
breaks our cbind "when to stop descendant+apply" heuristic and cbind
cannot be moved there as it'd be only copied over by initComponent
and that happens later than the cbind apply step.

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

pbs: drop unused reference

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

pbs: top bar UX improvements

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

pbs: refactor remote edit window

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

pbs: make remote add fully declarative

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

pbs: actually use cbind mixin for it to work

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

pbs: adapt to API changes + some litte cleanups

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

restore window: adapt for api changes

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

pbs: drop beta notice and encryption checkbox

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

add PBSConfig tab to Backup menu

The PBSConfig panel enables creation/editing/deletion of PBS instances.
Each instance can lists its snapshots and each snapshot can be restored

Inspired by the LDAPConfig panel and PBSEdit from pve-manager.

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

refactor RestoreWindow for PBS

by moving the item definition to initComponent, and changing the check
for a provided filename, we can reuse the window for restores from PBS

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

Make Backup/Restore panel a menuentry

Move it away from the tab list in the Configuration entry to a submenu in
preparation for adding PBS integration

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

RegexTester: fix getting reference to regex field

the regular expression field is not necessarily after the regex tester
(mostly it is before). In order to be more robust, use the child
method of the parent window instead of previousSibling of the button.

The child method gets any direct descendant matching the query, the
sibling one is always directional and there's no method for "any
sibling".

Tested with:
* mailproxy whitelist
* match field
* match filename
* who object

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

buildsys: rm -f .lint-incremental

else the build fails if the file does not exist

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

bump version to 2.2-3

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

mobile: consistent success action message with full UI

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

run lint checks during build and allow incremental linting

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

mobile: fix calendar start/end date labels

Since/Until is not quite correct, especially in other languages
(e.g., in German "from" and "to" was translated to "Von" and "An",
respectively.

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

Utils: add product specific task descriptions

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

mobile: show toast instead of annoying prompt

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

js/mobile: fix eslint and enable linting on build

lint the build file, as else eslint does not has the full picture.

Add a prelude to define globals once at the top

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

buildsys: use separate makefiles

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

eslint fixes

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

drop jslint hints

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

drop jslint hint for Proxmox global

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

tree wide: eslint --fix

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

quarantine action: use shadow for toast "window"

better contrast

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

buildsys: fix makefile dependencies

To ensure relevant stuff is always rebuild include the CSS and image
files in the install dependencies.

Further, mark the meta targets for building the pmgmanagerlib normal
and mobile versions as PHONY, as they only re-direct the make to the
respective makefile we need to ensure it always does, as else it
circumvents the actual dependency declaration of the responsible
targets.

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

quarantine: make annoying success prompt a toast

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

make lint: use eslint

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

utils: eslint fixes

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

fix #2844: MailTracker: do not collapse rows with double click

so that users can select words with double click, but can still expand
the logs with it

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

bump version to 2.2-2

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

postfix mailq admin: avoid early load by updating nodename only if changed

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

PostfixMailQ eslint fixes

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

deferred mailq: sort initially by arrival time

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

fix #1976: enable remoteSort for PostfixMailQueue

The mailq results are potentially very large, why a BufferedStore is used.

BufferedStores only support [0] remote filtering (which is already done) and
remote sorting (which this patch enables).

simpleSortMode (getting sortfield and sortdirection as two parameters instead
of an array with multiple objects containing those fields), is enabled to
work with our API.

[0] quite naturally, since you'd need to have the complete result to sort it,
 however only documented in the source code (and not the reference docs)

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

report: indentation cleanup

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

followup: add separator

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

gui: alow to remove subscription

Signed-off-by: Moayad Almalat <m.almalat@proxmox.com>

d/control: bump versioned to proxmox-widget-toolkit

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

remove Logo.js

it is now in the widget-toolkit

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

bump version to 2.2-1

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

MailTracker: change default emptyText

by default show that the user first has to press search, and
afterwards show the old emptyText

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

MailTracker: do not search when not both datetime parameters are given

in the gui, we always expect an end and starttime, the only case
where we do not get this is when the fields are invalid, so bail out
in that case instead of searching

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