Merge pull request #575 from jeremyandrews/sessions

test that per-user sessions are working
tag1consulting · Dec 21, 2023 · 00d5517 · 00d5517
2 parents b106feb + 31d88b5
commit 00d5517
Show file tree

Hide file tree

Showing 3 changed files with 444 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,7 @@
  - [#565](https://github.com/tag1consulting/goose/pull/565) add `--accept-invalid-certs` to skip validation of https certificates
  - [#568](https://github.com/tag1consulting/goose/pull/568) don't panic when truncating non utf-8 string
  - [#574](https://github.com/tag1consulting/goose/pull/574) update [`http`](https://docs.rs/http), [`itertools`](https://docs.rs/itertools) [`nix`](https://docs.rs/nix), [`rustls`](https://docs.rs/rustls/), and [`serial_test`](https://docs.rs/serial_test)
+ - [#575](https://github.com/tag1consulting/goose/pull/575) add test coverage for sessions and cookies, revert [#557](https://github.com/tag1consulting/goose/pull/557) to avoid sharing the CookieJar between all users
 
 ## 0.17.2 August 28, 2023
  - [#557](https://github.com/tag1consulting/goose/pull/557) speed up user initialization on Linux

diff --git a/src/lib.rs b/src/lib.rs
@@ -751,7 +751,6 @@ impl GooseAttack {
             self.test_plan.total_users()
         );
 
-        let reqwest_client = goose::create_reqwest_client(&self.configuration)?;
         let mut weighted_users = Vec::new();
         let mut user_count = 0;
         loop {
@@ -772,7 +771,7 @@ impl GooseAttack {
                     base_url,
                     &self.configuration,
                     self.metrics.hash,
-                    Some(reqwest_client.clone()),
+                    Some(goose::create_reqwest_client(&self.configuration)?),
                 )?);
                 user_count += 1;
                 if user_count == total_users {