Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: debug fetch of CRLs #272

Merged
merged 12 commits into from
Jul 15, 2024
Merged

test: debug fetch of CRLs #272

merged 12 commits into from
Jul 15, 2024

Conversation

Manuthor
Copy link
Contributor

@Manuthor Manuthor commented Jul 12, 2024

Du to reqwest/hyper error: connection closed before message completed:

  • to minimize bad reuse idle connection from reqwest pool connection, configure reqwest builder as follow :
            client: builder
                .connect_timeout(Duration::from_secs(10))
                .timeout(Duration::from_secs(10))
                .tcp_keepalive(Duration::from_secs(5))
                .pool_idle_timeout(Duration::from_secs(5))
                .pool_max_idle_per_host(2)
                .default_headers(headers)
                .build()?,
  • add retry strategy for getting CRL
  • regenerate certificates with CRL dependencies (make the URL in https)

@Manuthor Manuthor force-pushed the debug_getting_crls branch 4 times, most recently from 34735f5 to cdc21dd Compare July 13, 2024 05:46
@Manuthor Manuthor force-pushed the debug_getting_crls branch 2 times, most recently from 28a02e0 to 9f37c51 Compare July 13, 2024 08:34
@Manuthor Manuthor force-pushed the debug_getting_crls branch 3 times, most recently from f0a7a70 to bc73799 Compare July 13, 2024 20:57
@Manuthor Manuthor force-pushed the debug_getting_crls branch 2 times, most recently from 1598c0f to b13d655 Compare July 14, 2024 08:35
@Manuthor Manuthor requested a review from Adamk93 July 15, 2024 06:44
Copy link
Contributor

@Adamk93 Adamk93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems fine to me!

@Manuthor Manuthor merged commit cae8bc8 into develop Jul 15, 2024
35 checks passed
@Manuthor Manuthor deleted the debug_getting_crls branch July 15, 2024 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants