all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [pbs-devel] [PATCH proxmox-backup] cargo: require proxmox-auth-api 1.0.3
@ 2025-08-06  8:42 Shannon Sterz
  2025-08-06  9:56 ` Thomas Lamprecht
  2025-08-06 10:09 ` Thomas Lamprecht
  0 siblings, 2 replies; 5+ messages in thread
From: Shannon Sterz @ 2025-08-06  8:42 UTC (permalink / raw)
  To: pbs-devel

the only change is that we log the appropriate error return code on
failed login attempts.

Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>
---

would be nice to have that in there, as we seem to have a hard to
reproduce pam issue going on and that might give us some more
information.

 Cargo.toml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Cargo.toml b/Cargo.toml
index 637bf8f6..ee4952b3 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -56,7 +56,7 @@ path = "src/lib.rs"
 proxmox-apt = { version = "0.99", features = [ "cache" ] }
 proxmox-apt-api-types = "2"
 proxmox-async = "0.5"
-proxmox-auth-api = "1.0.2"
+proxmox-auth-api = "1.0.3"
 proxmox-base64 = "1"
 proxmox-borrow = "1"
 proxmox-compression = "1"
--
2.47.2



_______________________________________________
pbs-devel mailing list
pbs-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [pbs-devel] [PATCH proxmox-backup] cargo: require proxmox-auth-api 1.0.3
  2025-08-06  8:42 [pbs-devel] [PATCH proxmox-backup] cargo: require proxmox-auth-api 1.0.3 Shannon Sterz
@ 2025-08-06  9:56 ` Thomas Lamprecht
  2025-08-06 10:00   ` Shannon Sterz
  2025-08-06 10:09 ` Thomas Lamprecht
  1 sibling, 1 reply; 5+ messages in thread
From: Thomas Lamprecht @ 2025-08-06  9:56 UTC (permalink / raw)
  To: Proxmox Backup Server development discussion, Shannon Sterz

Am 06.08.25 um 10:43 schrieb Shannon Sterz:
> the only change is that we log the appropriate error return code on
> failed login attempts.
> 
> Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>
> ---
> 
> would be nice to have that in there, as we seem to have a hard to
> reproduce pam issue going on and that might give us some more
> information.

Latest PBS I uploaded is already build with that package though, so
this is just to ensure others build it also with this version, or?


_______________________________________________
pbs-devel mailing list
pbs-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [pbs-devel] [PATCH proxmox-backup] cargo: require proxmox-auth-api 1.0.3
  2025-08-06  9:56 ` Thomas Lamprecht
@ 2025-08-06 10:00   ` Shannon Sterz
  2025-08-06 10:08     ` Thomas Lamprecht
  0 siblings, 1 reply; 5+ messages in thread
From: Shannon Sterz @ 2025-08-06 10:00 UTC (permalink / raw)
  To: Thomas Lamprecht, Proxmox Backup Server development discussion

On Wed Aug 6, 2025 at 11:56 AM CEST, Thomas Lamprecht wrote:
> Am 06.08.25 um 10:43 schrieb Shannon Sterz:
>> the only change is that we log the appropriate error return code on
>> failed login attempts.
>>
>> Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>
>> ---
>>
>> would be nice to have that in there, as we seem to have a hard to
>> reproduce pam issue going on and that might give us some more
>> information.
>
> Latest PBS I uploaded is already build with that package though, so
> this is just to ensure others build it also with this version, or?

ah sorry than this was a misunderstanding on my side, yep can be
ignored. sorry for the noise


_______________________________________________
pbs-devel mailing list
pbs-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [pbs-devel] [PATCH proxmox-backup] cargo: require proxmox-auth-api 1.0.3
  2025-08-06 10:00   ` Shannon Sterz
@ 2025-08-06 10:08     ` Thomas Lamprecht
  0 siblings, 0 replies; 5+ messages in thread
From: Thomas Lamprecht @ 2025-08-06 10:08 UTC (permalink / raw)
  To: Shannon Sterz, Proxmox Backup Server development discussion

Am 06.08.25 um 12:00 schrieb Shannon Sterz:
> On Wed Aug 6, 2025 at 11:56 AM CEST, Thomas Lamprecht wrote:
>> Am 06.08.25 um 10:43 schrieb Shannon Sterz:
>>> the only change is that we log the appropriate error return code on
>>> failed login attempts.
>>>
>>> Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>
>>> ---
>>>
>>> would be nice to have that in there, as we seem to have a hard to
>>> reproduce pam issue going on and that might give us some more
>>> information.
>>
>> Latest PBS I uploaded is already build with that package though, so
>> this is just to ensure others build it also with this version, or?
> 
> ah sorry than this was a misunderstanding on my side, yep can be
> ignored. sorry for the noise

It's still fine to do, this was just more for me to ensure I did
not overlooked anything.


_______________________________________________
pbs-devel mailing list
pbs-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [pbs-devel] [PATCH proxmox-backup] cargo: require proxmox-auth-api 1.0.3
  2025-08-06  8:42 [pbs-devel] [PATCH proxmox-backup] cargo: require proxmox-auth-api 1.0.3 Shannon Sterz
  2025-08-06  9:56 ` Thomas Lamprecht
@ 2025-08-06 10:09 ` Thomas Lamprecht
  1 sibling, 0 replies; 5+ messages in thread
From: Thomas Lamprecht @ 2025-08-06 10:09 UTC (permalink / raw)
  To: pbs-devel, Shannon Sterz

On Wed, 06 Aug 2025 10:42:46 +0200, Shannon Sterz wrote:
> the only change is that we log the appropriate error return code on
> failed login attempts.
> 
> 

Applied, thanks!

I also bumped the version in d/control so that (package) build fails early.

[1/1] cargo: require proxmox-auth-api 1.0.3
      commit: 00ba5ed8de04ff85a3e75c3c8b17eafb0a98f4db


_______________________________________________
pbs-devel mailing list
pbs-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2025-08-06 10:10 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-08-06  8:42 [pbs-devel] [PATCH proxmox-backup] cargo: require proxmox-auth-api 1.0.3 Shannon Sterz
2025-08-06  9:56 ` Thomas Lamprecht
2025-08-06 10:00   ` Shannon Sterz
2025-08-06 10:08     ` Thomas Lamprecht
2025-08-06 10:09 ` Thomas Lamprecht

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal