sprig icon indicating copy to clipboard operation
sprig copied to clipboard

derivePassword requires counter as uint32

Open AssafKatz3 opened this issue 6 years ago • 5 comments

Hi, I am trying to use derivePassword in loop but unfortunately counter is defined uint32 so it is hard (I didn't find any way to do it from HELM) to use it. Thanks N.B. The algorithm of this function moved to here

AssafKatz3 avatar Jul 23 '18 16:07 AssafKatz3

I have not had time to look into this. Do you have a suggested change and reason for it?

mattfarina avatar Nov 19 '18 19:11 mattfarina

@mattfarina As I wrote, the reason is making it useful inside HELM loop (range). About the change itself, I cannot suggest it since I don't know golang good enough. I am sure that derivePassword signture should be changed to int32. But I am less sure what other should be change.

AssafKatz3 avatar Nov 20 '18 06:11 AssafKatz3

+1. the only way to use this function is to hardcode the counter, which is a curious restriction if intentional.

dkrieger avatar Mar 08 '19 03:03 dkrieger

Sorry for reviving an old bug/thread but this is still an issue for those of us who would like to use atoi on strings as IDs for derrivePassword. Perhaps a type converstion function from int/int64 to uint32 would be sufficient?

antevens avatar Mar 23 '23 02:03 antevens

We still need it...

AssafKatz3 avatar Oct 11 '23 06:10 AssafKatz3