Stop printing message when doing a context shutdown

[rubenv]

Uses the configured logger instead.

[cristaloleg]

Good catch, thanks!

[cristaloleg]

Probably this was the reason :) https://github.com/allegro/bigcache/actions/runs/6145556141/job/16673759956?pr=374#step:5:25

[rubenv]

I honestly have no idea why it would cause that.

[cristaloleg]

AFAIR it's because during a test run os.Stdout is replaced with a test-specific writer to catch the full output. And because this we're logging in a goroutine we can log after a test end, which results in a panic due to Go test cleanup.

If you're curious to fix it than try to replace this context with context.WithCancel(...)

bigcache/examples_test.go

Line 63 in 157556c

cache, initErr := bigcache.New(context.Background(), config)

And if you're not curious (which is fine too) I will take a look later today (leaving PR open for some time).

Thanks.

[janisz]

This will solve the data race

Suggested change

	go func() {
	logger := newLogger(config.Logger)
	logger := newLogger(config.Logger)
	go func() {

[cristaloleg]

Nice! anyway, replacing ctx is also a good thing for the future.

[janisz]

@rubenv Ping