public inbox for pbs-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Lukas Wagner <l.wagner@proxmox.com>
To: pve-devel@lists.proxmox.com, pbs-devel@lists.proxmox.com
Subject: [pbs-devel] [PATCH proxmox{, -perl-rs, -backup} 0/5] move `HttpError` from `proxmox-router` into its own crate
Date: Wed, 26 Jul 2023 14:50:01 +0200	[thread overview]
Message-ID: <20230726125006.616124-1-l.wagner@proxmox.com> (raw)

This patch series moves `HttpError` from `proxmox-router` into its own little 
crate. The main benefit of this is that one can now use `HttpError`
independently (e.g. for `proxmox-notify`)

These patches also update the users of the old `HttpError` to the new version.
`proxmox-notify`, which previously used it's own error type, is now also using 
the new crate.



proxmox:

Lukas Wagner (3):
  http-error: add new http-error crate
  router: rest-server: auth-api: use new http-error crate
  notify: use HttpError from proxmox-http-error

 Cargo.toml                           |   2 +
 proxmox-auth-api/Cargo.toml          |   2 +
 proxmox-auth-api/src/api/access.rs   |   3 +-
 proxmox-http-error/Cargo.toml        |  16 ++++
 proxmox-http-error/src/lib.rs        |  81 +++++++++++++++++
 proxmox-notify/Cargo.toml            |   1 +
 proxmox-notify/src/api/common.rs     |  33 ++++---
 proxmox-notify/src/api/filter.rs     |  59 ++++++------
 proxmox-notify/src/api/gotify.rs     |  85 ++++++++++--------
 proxmox-notify/src/api/group.rs      |  82 +++++++++--------
 proxmox-notify/src/api/mod.rs        | 130 ++++++++++-----------------
 proxmox-notify/src/api/sendmail.rs   |  93 ++++++++++---------
 proxmox-rest-server/Cargo.toml       |   1 +
 proxmox-rest-server/src/h2service.rs |   2 +-
 proxmox-rest-server/src/rest.rs      |   2 +-
 proxmox-router/Cargo.toml            |   1 +
 proxmox-router/src/error.rs          |  44 ---------
 proxmox-router/src/lib.rs            |   7 +-
 18 files changed, 340 insertions(+), 304 deletions(-)
 create mode 100644 proxmox-http-error/Cargo.toml
 create mode 100644 proxmox-http-error/src/lib.rs
 delete mode 100644 proxmox-router/src/error.rs


proxmox-perl-rs:

Lukas Wagner (1):
  notify: use new HttpError type

 common/src/notify.rs | 77 +++++++++++++++++++++++---------------------
 pve-rs/Cargo.toml    |  1 +
 2 files changed, 42 insertions(+), 36 deletions(-)


proxmox-backup:

Lukas Wagner (1):
  use `HttpError` and macros from `proxmox-http-error` crate

 Cargo.toml                              | 3 +++
 src/api2/access/openid.rs               | 5 ++---
 src/api2/access/tfa.rs                  | 3 ++-
 src/api2/admin/datastore.rs             | 5 +++--
 src/api2/admin/namespace.rs             | 3 ++-
 src/api2/backup/mod.rs                  | 5 +++--
 src/api2/config/access/ldap.rs          | 3 ++-
 src/api2/config/access/openid.rs        | 3 ++-
 src/api2/config/acme.rs                 | 5 ++---
 src/api2/config/changer.rs              | 3 ++-
 src/api2/config/datastore.rs            | 3 ++-
 src/api2/config/drive.rs                | 3 ++-
 src/api2/config/media_pool.rs           | 3 ++-
 src/api2/config/prune.rs                | 3 ++-
 src/api2/config/remote.rs               | 3 ++-
 src/api2/config/sync.rs                 | 3 ++-
 src/api2/config/tape_backup_job.rs      | 3 ++-
 src/api2/config/tape_encryption_keys.rs | 3 ++-
 src/api2/config/traffic_control.rs      | 3 ++-
 src/api2/config/verify.rs               | 3 ++-
 src/api2/helpers.rs                     | 2 +-
 src/api2/reader/mod.rs                  | 5 +++--
 src/auth.rs                             | 2 +-
 src/backup/hierarchy.rs                 | 2 +-
 src/server/pull.rs                      | 4 ++--
 25 files changed, 51 insertions(+), 32 deletions(-)


Summary over all repositories:
  45 files changed, 433 insertions(+), 372 deletions(-)

-- 
murpp v0.4.0





             reply	other threads:[~2023-07-26 12:50 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-07-26 12:50 Lukas Wagner [this message]
2023-07-26 12:50 ` [pbs-devel] [PATCH proxmox 1/5] http-error: add new http-error crate Lukas Wagner
2023-07-26 13:35   ` Wolfgang Bumiller
2023-07-26 12:50 ` [pbs-devel] [PATCH proxmox 2/5] router: rest-server: auth-api: use " Lukas Wagner
2023-07-26 13:41   ` Wolfgang Bumiller
2023-07-26 13:45     ` Lukas Wagner
2023-07-26 12:50 ` [pbs-devel] [PATCH proxmox 3/5] notify: use HttpError from proxmox-http-error Lukas Wagner
2023-07-26 12:50 ` [pbs-devel] [PATCH proxmox-perl-rs 4/5] notify: use new HttpError type Lukas Wagner
2023-07-26 12:50 ` [pbs-devel] [PATCH proxmox-backup 5/5] use `HttpError` and macros from `proxmox-http-error` crate Lukas Wagner
2023-07-26 13:42   ` Wolfgang Bumiller

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230726125006.616124-1-l.wagner@proxmox.com \
    --to=l.wagner@proxmox.com \
    --cc=pbs-devel@lists.proxmox.com \
    --cc=pve-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal