search

smocker-dev / smocker

Smocker is a simple and efficient HTTP mock server and proxy
https://smocker.dev
MIT License
1.13k stars 59 forks source link

use atomic to fix mock timesCount counter #273

Closed SuddenGunter closed 9 months ago

SuddenGunter commented 1 year ago

Fix for the issue https://github.com/Thiht/smocker/issues/272

One of the issues I see in my test, that it generates a lot of logs, but I can't disable it without unrelated changes in server.NewMockServer

SuddenGunter commented 1 year ago

@Thiht would you have a time to review this pls?

gwleclerc commented 9 months ago

Thanks for the PR but solved using an other mutex in #285 to maintain a consistent approach