chunk_store: Fix typo in bail message

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>

backup: Fix typos and grammar

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>

docs: fix #2851 Add note about GC grace period

Adding a note about the garbage collection's grace period due to the
default atime behavior should help to avoid confusion as to why space is
not freed immediately.

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>

followup: fixup the directory number, refactor

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

docs: add note on supported filesystems

certain filesystems cannot be used as chunkstores, because they don't
support 2^16 subdirectories (e.g. ext4 with certain features disabled
or ext3 - see ext4(5))

reported via our community forum:
https://forum.proxmox.com/threads/emlink-too-many-links.73108/

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

docs: introduction: rewording and fixing of minor errors

Reworded one sentence for improved readability.
Fixed some minor language errors.

Signed-off-by: Dylan Whyte <d.whyte@proxmox.com>

docs: admin guide: fix grammatical errors and improve English

Mostly fixed typos and grammatical errors.
Improved wording in some sections to make instructions/advice clearer.

Signed-off-by: Dylan Whyte <d.whyte@proxmox.com>

bump udev build-dependency

0.4 contains a fix for C chars on non-x86 architectures.

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

build: run tests on build (again)

now that all examples and tests are fixed again.

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

manifest: revert canonicalization to old behaviour

JSON keys MUST be quoted. this is a one-time break in signature
validation for backups created with the broken canonicalization code.
QEMU backups are not affected, as libproxmox-backup-qemu never linked
the broken versions.

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

examples/upload-speed: adapt to change

commit 323b2f3dd6364119f2ffcd7397161e9f56cdb3fc
changed the signature of upload_speedtest
adapt the example

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

tools/xattr: a char from C is not universally a rust i8

Make it actually do the correct cast by using `libc::c_char`.

Fixes issues when building on other platforms, e.g., the aarch64
client only build on Arch Linux ARM I tested in my free time.

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

server: add path value to NOT_FOUND http error

Especially helpful for requests not coming from browsers (where the
URL is normally easy to find out).

Makes it easier to detect if one triggered a request with an old
client, or so..

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

bump pxar dependency to 0.2.1

Contains a fix for the check for the maximum allowed size of
acl group object entries.

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

catalog_shell: add exit command

it is nice to have a command to exit from the shell instead of
only allowing ctrl+d or ctrl+c

the api method is just for documentation/help purposes and does nothing
by itself, the real logic is directly in the read loop

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

pxar: .pxarexclude: absolute paths and byte based paths

Change the .pxarexclude parser to byte based parsing with
`.split(b'\n')` instead of `.lines()`, to not panic on
non-utf8 paths.

Specially deal with absolute paths by prefixing them with
the current directory.

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

tools: add strip_ascii_whitespace for byte slices

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

cached user info: check_privs: print privilege path in error message

As else this is really user unfriendly, and it not printing it has no
advantage. If one doesn't wants to leak resource existence they just
need to *always* check permissions before checking if the requested
resource exists, if that's not done one can leak information also
without getting the path returned (as the system will either print
"resource doesn't exists" or "no permissions" respectively)

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

buildsys: fix targets to not run dpkg-buildpackage 4 times

and add a deb-all target

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

bump version to 0.8.7-2

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

d/lintian-overrides: ignore systemd-service-file-refers-to-unusual-wantedby-target

proxmox-backup-banner.service needs getty.target

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

ui: datastore edit: avoid an extra indentation level

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

warning fixup

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

bump version to 0.8.7-1

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

pxar: restore file attributes, improve errors

and use the correct integer types for these operations

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

ui: buildsys: add lint target

not yet automatically called on build, as it still fails.

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

docs: installation: add note where to find the webinterface

As the 8007 vs 8006 port is new and could confuse people, especially
if they did not used the PBS installer.

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

docs: fix typo s/PBS_REPOSTOR/PBS_REPOSITOR/

Reported-by: Piotr Paszkowski aka patefoniQ
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

ui: running tasks: update limit to 100

else we'll never see the 99+ tasks ..

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

docs: add secure-apt

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

bump version to 0.8.6-1

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

src/backup/data_blob_reader.rs: avoid unwrap() - return error instead

ui: fingerprint: add icon to copy button

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

ui: fingerprint: add copy button

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

ui: add show fingerprint button to dashboard

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

api2/node/status: add fingerprint

and rename get_usage to get_status (since its not usage only anymore)

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

refactor CertInfo to tools

we want to reuse some of the functionality elsewhere

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

update proxmox crate to 0.2.0

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

src/bin/proxmox_backup_client/catalog.rs: fix keyfile handling

src/backup/manifest.rs: fix signature (exclude 'signature' property)

client: show key path when creating/changing default key

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

client: add simple version command

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

proxmox-backup-client benchmark: improve output format

src/server/rest.rs: avoid compiler warning

ui: running task: increase active limit we show in badge to 99

Two digits fit nicely, and the extra plus for the >99 case doesn't
takes that much space either. So that and the fact that 9 is just
really low makes me bump this to 99 as cut-off value.

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

ui: tune badge styling a bit

the idea is to blend in when no task is running, thus no
background-color there. When tasks are running use the proxmox
branding guideline dark-grey, it isn't used as often so it should
fall into ones eye when changing but it has some use so it doesn't
seems out of place.

Reduce the border radius by a lot, so that it seems similar to the
one our ExtJS theme uses for the buttons outside - the original
border radius seems like it comes from the time where this was
intended to be a floating badge, there it'd make sense but as
integrated button one this seems to fit the style much more.

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

docs: reference PDF variant in HTML output

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

debian/proxmox-backup-docs.links: fix name and target

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

datastore: chown base dir on creation

When creating a new datastore the basedir is only owned by the backup
user if it did not exist beforehand (create_path chowns only if it
creates the directory), and returns false if it did not create the
directory).

This improves the experience when adding a new datastore on a fresh
disk or existing directory (not owned by backup) - backups/pulls can
be run instead of terminating with EPERM.

Tested on my local testinstall with a new disk, and a existing directory:

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

ui: dashboard/TaskSummary: fade icons if count is zero

so that users can see the relevant counts faster

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

ui: fix crypt mode caluclation

also include 'mixed' in the calculation of the overall mode of a
snapshot and group

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

docs: add initial basic software stack definition

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

proxmox-backup-client benchamrk: add more speed tests

src/server/rest.rs: disable debug logs

proxmox-backup-client benchmark: add --verbose flag

bump version to 0.8.5-1

fix cross process task listing

it does not make sense to check if the worker is running if we already
have an endtime and state

our 'worker_is_active_local' heuristic returns true for non
process-local tasks, so we got 'running' for all tasks that were not
started by 'our' pid and were still running

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

docs: expand datastore documentation

document retention settings and schedules per datastore with
some minimal examples.

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

docs: add remotes and sync-jobs and schedules

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

bump pathpatterns to 0.1.2

Fixes `**/foo` not matching "foo" without slashes.
(`**/lost+found` now matches the `lost+found` dir at the
root of our tree properly).

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

ui: align version and user-menu spacing with pve/pmg

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

ui: make username a menu-button

like we did in PVE and PMG

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

bump version to 0.8.4-1

ui: add TaskButton in header

opens a grid with the running tasks and a shortcut the the node tasks

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

ui: add RunningTasksStore

so that we have a global store for running tasks

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

simpler lost+found pattern

the **/ is not required and currently also mistakenly
doesn't match /lost+found which is probably buggy on the
pathpatterns crate side and needs fixing there

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

bump version to 0.8.3-1

get_disks: don't fail on zfs_devices

zfs does not have to be installed, so simply log an error and
continue, users still get an error when clicking directly on
ZFS

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

disks/zpool_status: add test for pool with special character

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

disks/zpool_list: allow some more characters for pool list

not exhaustive of what zfs allows (space is missing), but this
can be done easily without problems

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

zpool_list: add tests for special pool names

those names are allowed for zpools

these will fail for now, but it will be fixed in the next commit

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

api: add ZPOOL_NAME_SCHEMA and regex

poolnames can containe spaces and some other special characters

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

minor optimization to 'to_canonical_json'

* don't clone hash keys, just use references
* we don't need a String, stick to Vec<u8> and use
  serde_json::to_writer to avoid a temporary strings
  altogether

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

ui: adapt for new sign-only crypt mode

we can now show 'none', 'encprypted', 'signed' or 'mixed' for
the crypt mode

also adds a different icon for signed files, and adds a hint that
signatures cannot be verified on the server

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

src/backup/manifest.rs: add default toömake it compatible with older backus

bump version toö 0.8.2-1

buildsys: also upload debug packages

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

src/backup/manifest.rs: rename into_string -> to_string

And do not consume self.

bump version to 0.8.1-1

src/backup/manifest.rs: use serde_json::from_value() to deserialize data

Also modified from_data compute signature ditectly from json.

typos

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

src/backup/manifest.rs: cleanup signature generation

tests/blob_writer.rs: make it work again

ship pbstest repo as sources.list.d file for beta

NOTE: the repo url is not yet working at time of commit, this is a
preparatory step.

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

docs: package repos: drop non-tests for now

they won't work and thus just confuse people, re-add them once we're
releasing final.

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

docs: move host sysadmin out to own chapter, fix ZFS one

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

docs: add missing todos

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

add local-zfs.rst

content is > 90% same as local-zfs.adoc in pve-docs.

adapted the format for .rst

fixed some typos and wrote some parts slightly different (wording).

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

src/backup/manifest.rs: include signature inside the manifest

This is more flexible, because we can choose what fileds we want to sign.

src/backup/file_formats.rs: remove signed chunks

We can include signature in the manifest instead (patch will follow).

docs: fix repo paths

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

improve administration guide

fixing some typos and grammar errors.

added example file layout for datastores.

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

client: introduce --keyfd parameter

This is a more convenient way to pass along the key when
creating encrypted backups of unprivileged containers in PVE
where the unprivileged user namespace cannot access
`/etc/pve/priv`.

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

src/backup/crypt_config.rs - compute_digest: make it more secure

ui: ds/content: disable some button for in-progress backup

We cannot verify, download, file-browse backups which are currently
in progress.

'Forget' could work but is probably not desirable?

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

ui: ds/content: do not count in-progress backups for last made one

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

ui: ds/content: show spinner for backups in progress

use the fact that they do not have a size property at all

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

more xdg cleanup and encryption parameter improvements

Have a single common function to get the BaseDirectories
instance and a wrapper for `find()` and `place()` which
wrap the error with some context.

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

client: xdg usage: place() vs find()

place() is used when creating a file, as it will create
intermediate directories, only use it when actually placing
a new file.

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

src/client/backup_writer.rs - upload_stream: add crypt_mode