Some checks failed
CI / Run tests and lints (push) Failing after 29s
CI / Build CLI docs (push) Has been skipped
CI / Run Playwright end-to-end tests (mysql) (push) Failing after 3s
CI / Run Playwright end-to-end tests (postgres) (push) Failing after 2s
CI / Run Playwright end-to-end tests (sqlite) (push) Failing after 3s
CodeQL / Analyze (go) (push) Failing after 4s
CodeQL / Analyze (javascript) (push) Failing after 3s
Docker Image Scanners / scanners (push) Failing after 4s
Synchronize Issue Labels / Synchronize Issue Labels (push) Has been skipped
CI / Run end-to-end tests (postgres) (push) Failing after 19s
CI / Run end-to-end tests (sqlite) (push) Failing after 2s
CI / Run Playwright end-to-end tests (cockroach) (push) Failing after 2s
CI / Draft newsletter (push) Has been skipped
CI / Generate release (push) Has been skipped
CI / Pending approval Slack notification (push) Has been skipped
CI / Send newsletter (push) Has been skipped
333 lines
11 KiB
Go
333 lines
11 KiB
Go
// Copyright © 2023 Ory Corp
|
|
// SPDX-License-Identifier: Apache-2.0
|
|
|
|
package argon2
|
|
|
|
import (
|
|
"fmt"
|
|
"io"
|
|
"runtime"
|
|
"strconv"
|
|
"time"
|
|
|
|
"github.com/ory/x/configx"
|
|
|
|
"github.com/ory/x/cmdx"
|
|
|
|
"github.com/fatih/color"
|
|
"github.com/inhies/go-bytesize"
|
|
"github.com/spf13/cobra"
|
|
|
|
"github.com/ory/kratos/driver/config"
|
|
"github.com/ory/kratos/hash"
|
|
)
|
|
|
|
const (
|
|
FlagStartMemory = "start-memory"
|
|
FlagMaxMemory = "max-memory"
|
|
FlagAdjustMemory = "adjust-memory-by"
|
|
FlagStartIterations = "start-iterations"
|
|
FlagMaxConcurrent = "max-concurrent"
|
|
|
|
FlagRuns = "probe-runs"
|
|
)
|
|
|
|
type (
|
|
colorWriter struct {
|
|
c *color.Color
|
|
w io.Writer
|
|
}
|
|
loadResult struct {
|
|
r *resultTable
|
|
v *config.Argon2
|
|
}
|
|
loadResults []*loadResult
|
|
)
|
|
|
|
var _ cmdx.Table = loadResults{}
|
|
|
|
func (l loadResults) Header() []string {
|
|
return append((&resultTable{}).Header(), "MEMOry PARAM", "ITERATIONS PARAM")
|
|
}
|
|
|
|
func (l loadResults) Table() [][]string {
|
|
t := make([][]string, len(l))
|
|
|
|
for i, r := range l {
|
|
t[i] = append(r.r.Columns(), fmt.Sprintf("%d", r.v.Memory), fmt.Sprintf("%d", r.v.Iterations))
|
|
}
|
|
|
|
return t
|
|
}
|
|
|
|
func (l loadResults) Interface() interface{} {
|
|
return l
|
|
}
|
|
|
|
func (l loadResults) Len() int {
|
|
return len(l)
|
|
}
|
|
|
|
func (c *colorWriter) Write(o []byte) (int, error) {
|
|
return c.c.Fprintf(c.w, "%s", o)
|
|
}
|
|
|
|
func newCalibrateCmd() *cobra.Command {
|
|
var (
|
|
maxMemory, adjustMemory bytesize.ByteSize = 0, 512 * bytesize.MB
|
|
runs int
|
|
)
|
|
|
|
flagConfig := &argon2Config{
|
|
localConfig: config.Argon2{},
|
|
}
|
|
|
|
cmd := &cobra.Command{
|
|
Use: "calibrate <requests-per-minute>",
|
|
Args: cobra.ExactArgs(1),
|
|
Short: "Computes Optimal Argon2 Parameters",
|
|
Long: `This command helps you calibrate the configuration parameters for Argon2. Password hashing is a trade-off between security, resource consumption, and user experience. Resource consumption should not be too high and the login should not take too long.
|
|
|
|
We recommend that the login process takes between half a second and one second for password hashing, giving a good balance between security and user experience.
|
|
|
|
Please note that the values depend on the machine you run the hashing on. If you have RAM constraints, please set the memory dedicated to Ory Kratos to avoid out of memory panics.`,
|
|
RunE: func(cmd *cobra.Command, args []string) error {
|
|
progressPrinter := cmdx.NewLoudErrPrinter(cmd)
|
|
resultPrinter := cmdx.NewLoudPrinter(cmd, &colorWriter{c: color.New(color.FgGreen), w: cmd.ErrOrStderr()})
|
|
|
|
conf, err := configProvider(cmd, flagConfig)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
// always take start flags, or their default
|
|
conf.localConfig.Memory = flagConfig.localConfig.Memory
|
|
conf.localConfig.Iterations = flagConfig.localConfig.Iterations
|
|
|
|
desiredDuration := conf.localConfig.ExpectedDuration
|
|
|
|
reqPerMin, err := strconv.ParseInt(args[0], 0, 0)
|
|
if err != nil {
|
|
// we want the error and usage string printed so just return
|
|
return err
|
|
}
|
|
|
|
hasher := hash.NewHasherArgon2(conf)
|
|
|
|
var currentDuration time.Duration
|
|
|
|
_, _ = progressPrinter.Printf("Increasing memory to get over %s:\n", desiredDuration)
|
|
|
|
for {
|
|
if maxMemory != 0 && conf.localConfig.Memory > maxMemory {
|
|
// don't further increase memory
|
|
_, _ = progressPrinter.Println(" ouch, hit the memory limit there")
|
|
conf.localConfig.Memory = maxMemory
|
|
break
|
|
}
|
|
|
|
currentDuration, err = probe(cmd, hasher, runs, progressPrinter)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
_, _ = progressPrinter.Printf(" took %s with %s of memory\n", currentDuration, conf.localConfig.Memory)
|
|
|
|
if currentDuration > desiredDuration {
|
|
if conf.localConfig.Memory <= adjustMemory {
|
|
// adjusting the memory would now result in <= 0B
|
|
adjustMemory = adjustMemory >> 1
|
|
}
|
|
conf.localConfig.Memory -= adjustMemory
|
|
break
|
|
}
|
|
|
|
// adjust config
|
|
conf.localConfig.Memory += adjustMemory
|
|
}
|
|
|
|
_, _ = progressPrinter.Printf("Decreasing memory to get under %s:\n", desiredDuration)
|
|
|
|
for {
|
|
currentDuration, err = probe(cmd, hasher, runs, progressPrinter)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
_, _ = progressPrinter.Printf(" took %s with %s of memory\n", currentDuration, conf.localConfig.Memory)
|
|
|
|
if currentDuration < desiredDuration {
|
|
break
|
|
}
|
|
|
|
for conf.localConfig.Memory <= adjustMemory {
|
|
// adjusting the memory would now result in <= 0B
|
|
adjustMemory = adjustMemory >> 1
|
|
}
|
|
|
|
// adjust config
|
|
conf.localConfig.Memory -= adjustMemory
|
|
}
|
|
|
|
_, _ = resultPrinter.Printf("Settled on %s of memory.\n", conf.localConfig.Memory)
|
|
_, _ = progressPrinter.Printf("Increasing iterations to get over %s:\n", desiredDuration)
|
|
|
|
for {
|
|
currentDuration, err = probe(cmd, hasher, runs, progressPrinter)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
_, _ = progressPrinter.Printf(" took %s with %d iterations\n", currentDuration, conf.localConfig.Iterations)
|
|
|
|
if currentDuration > desiredDuration {
|
|
if conf.localConfig.Iterations > 1 {
|
|
conf.localConfig.Iterations -= 1
|
|
}
|
|
break
|
|
}
|
|
|
|
// adjust config
|
|
conf.localConfig.Iterations += 1
|
|
}
|
|
|
|
_, _ = progressPrinter.Printf("Decreasing iterations to get under %s:\n", desiredDuration)
|
|
|
|
for {
|
|
currentDuration, err = probe(cmd, hasher, runs, progressPrinter)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
_, _ = progressPrinter.Printf(" took %s with %d iterations\n", currentDuration, conf.localConfig.Iterations)
|
|
|
|
// break also when iterations is 1; this catches the case where 1 was only slightly under the desired time and took longer a bit longer on another run
|
|
if currentDuration < desiredDuration || conf.localConfig.Iterations == 1 {
|
|
break
|
|
}
|
|
|
|
// adjust config
|
|
conf.localConfig.Iterations -= 1
|
|
}
|
|
_, _ = resultPrinter.Printf("Settled on %d iterations.\n", conf.localConfig.Iterations)
|
|
|
|
results := make(loadResults, 5)
|
|
for i := 0; i < len(results); i++ {
|
|
_, _ = progressPrinter.Printf("\nRunning load test to simulate %d login requests per minute.\n", reqPerMin)
|
|
|
|
res, err := runLoadTest(cmd, conf, int(reqPerMin))
|
|
if err != nil {
|
|
return err
|
|
}
|
|
cv, _ := conf.HasherArgon2()
|
|
ccv := *cv
|
|
results[i] = &loadResult{r: res, v: &ccv}
|
|
|
|
_, _ = progressPrinter.Println("The load test result is:")
|
|
_, _ = progressPrinter.Println()
|
|
cmdx.PrintRow(cmd, res)
|
|
|
|
switch {
|
|
// too fast
|
|
case res.MedianTime < conf.localConfig.ExpectedDuration:
|
|
_, _ = progressPrinter.Printf("The median was %s under the minimal duration of %s, going to increase the hash cost.\n", conf.localConfig.ExpectedDuration-res.MedianTime, conf.localConfig.ExpectedDuration)
|
|
|
|
// try to increase memory first
|
|
if res.MaxMem+64*bytesize.MB < maxMemory {
|
|
// only small amounts of memory are sensible as we already benchmarked a single, non-concurrent request
|
|
conf.localConfig.Memory += 64 * bytesize.MB
|
|
_, _ = progressPrinter.Printf("Increasing memory to %s\n", conf.localConfig.Memory)
|
|
} else {
|
|
// increasing memory is not allowed by maxMemory, therefore increase CPU load
|
|
conf.localConfig.Iterations++
|
|
_, _ = progressPrinter.Printf("Increasing iterations to %d\n", conf.localConfig.Iterations)
|
|
}
|
|
// too much memory
|
|
case res.MaxMem > conf.localConfig.DedicatedMemory:
|
|
_, _ = progressPrinter.Printf("The required memory was %s more than the maximum allowed of %s.\n", res.MaxMem-maxMemory, conf.localConfig.DedicatedMemory)
|
|
|
|
//nolint:gosec // disable G115
|
|
conf.localConfig.Memory -= (res.MaxMem - conf.localConfig.DedicatedMemory) / bytesize.ByteSize(reqPerMin)
|
|
_, _ = progressPrinter.Printf("Decreasing memory to %s\n", conf.localConfig.Memory)
|
|
// too slow
|
|
case res.MaxTime > conf.localConfig.ExpectedDeviation+conf.localConfig.ExpectedDuration:
|
|
_, _ = progressPrinter.Printf("The longest request took %s longer than the longest acceptable time of %s, going to decrease the hash cost.\n", res.MaxTime-conf.localConfig.ExpectedDeviation+conf.localConfig.ExpectedDuration, conf.localConfig.ExpectedDeviation+conf.localConfig.ExpectedDuration)
|
|
|
|
// try to decrease iterations first
|
|
if conf.localConfig.Iterations > 1 {
|
|
conf.localConfig.Iterations--
|
|
_, _ = progressPrinter.Printf("Decreasing iterations to %d\n", conf.localConfig.Iterations)
|
|
} else {
|
|
// decreasing iterations is not possible anymore, decreasing memory
|
|
// only small amounts of memory are sensible as we already benchmarked a single, non-concurrent request
|
|
conf.localConfig.Memory -= 64 * bytesize.MB
|
|
_, _ = progressPrinter.Printf("Decreasing memory to %s\n", conf.localConfig.Memory)
|
|
}
|
|
// too high deviation
|
|
case res.StdDev > conf.localConfig.ExpectedDeviation:
|
|
_, _ = progressPrinter.Printf("The deviation was %s more than the expected deviation of %s.\n", res.StdDev-conf.localConfig.ExpectedDeviation, conf.localConfig.ExpectedDeviation)
|
|
|
|
// try to decrease iterations first
|
|
if conf.localConfig.Iterations > 1 {
|
|
conf.localConfig.Iterations--
|
|
_, _ = progressPrinter.Printf("Decreasing iterations to %d\n", conf.localConfig.Iterations)
|
|
} else {
|
|
// decreasing iterations is not possible anymore, decreasing memory
|
|
// only small amounts of memory are sensible as we already benchmarked a single, non-concurrent request
|
|
conf.localConfig.Memory -= 64 * bytesize.MB
|
|
_, _ = progressPrinter.Printf("Decreasing memory to %s\n", conf.localConfig.Memory)
|
|
}
|
|
// all values seem reasonable
|
|
default:
|
|
_, _ = progressPrinter.Println("These values look good to me.")
|
|
_, _ = progressPrinter.Println()
|
|
cmdx.PrintRow(cmd, conf)
|
|
return nil
|
|
}
|
|
}
|
|
|
|
_, _ = fmt.Fprintln(cmd.ErrOrStderr(), "Could not automatically determine good parameters. Have a look at all the measurements taken and select acceptable values yourself. Have a look in the docs for more information: https://www.ory.com/kratos/docs/debug/performance-out-of-memory-password-hashing-argon2")
|
|
cmdx.PrintTable(cmd, results)
|
|
return nil
|
|
},
|
|
}
|
|
|
|
flags := cmd.Flags()
|
|
|
|
flags.IntVarP(&runs, FlagRuns, "r", 2, "Runs per probe, median of all runs is taken as the result.")
|
|
|
|
flags.VarP(&flagConfig.localConfig.Memory, FlagStartMemory, "m", "Amount of memory to start probing at.")
|
|
flags.Var(&maxMemory, FlagMaxMemory, "Maximum memory allowed (0 means no limit).")
|
|
flags.Var(&adjustMemory, FlagAdjustMemory, "Amount by which the memory is adjusted in every step while probing.")
|
|
|
|
flags.Uint32VarP(&flagConfig.localConfig.Iterations, FlagStartIterations, "i", 1, "Number of iterations to start probing at.")
|
|
|
|
flags.Uint8(FlagMaxConcurrent, 16, "Maximum number of concurrent hashing operations.")
|
|
|
|
registerArgon2ConstantConfigFlags(flags, flagConfig)
|
|
cmdx.RegisterFormatFlags(flags)
|
|
configx.RegisterFlags(flags)
|
|
|
|
return cmd
|
|
}
|
|
|
|
func probe(cmd *cobra.Command, hasher hash.Hasher, runs int, progressPrinter *cmdx.ConditionalPrinter) (time.Duration, error) {
|
|
// force GC at the start of the experiment
|
|
runtime.GC()
|
|
|
|
start := time.Now()
|
|
|
|
var mid time.Time
|
|
for i := 0; i < runs; i++ {
|
|
mid = time.Now()
|
|
_, err := hasher.Generate(cmd.Context(), []byte("password"))
|
|
if err != nil {
|
|
_, _ = fmt.Fprintf(cmd.ErrOrStderr(), "Could not generate a hash: %s\n", err)
|
|
return 0, cmdx.FailSilently(cmd)
|
|
}
|
|
|
|
_, _ = progressPrinter.Printf(" took %s in try %d\n", time.Since(mid), i)
|
|
}
|
|
return time.Duration(int64(time.Since(start)) / int64(runs)), nil
|
|
}
|