public inbox for pbs-devel@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 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