fix(autotls): handle context cancellation during sleep intervals in c…#566
Merged
gacevicljubisa merged 4 commits intofeat/p2p-wss-supportfrom Feb 17, 2026
Merged
fix(autotls): handle context cancellation during sleep intervals in c…#566gacevicljubisa merged 4 commits intofeat/p2p-wss-supportfrom
gacevicljubisa merged 4 commits intofeat/p2p-wss-supportfrom
Conversation
gacevicljubisa
added a commit
that referenced
this pull request
Mar 31, 2026
* feat: add new flags * fix(k8s): ensure p2p-wss service port is created when configured * chore: add comments to local.yaml * fix(config): update autotls-registration-endpoint * feat(autotls): add autotls support with pebble and p2p-forge (#559) * feat(autotls): add auto tls support in init containers and configuration options * fix(k8s): fix EnvVar handling by adding hasValues method and improving toK8S conversion * feat(config): add p2p-wss support and update autotls registration endpoint in configuration files * chore(config): remove unneeded p2p-wss settings * feat(autotls): implement autotls check functionality (#560) * feat(autotls): implement autotls check functionality * feat(autotls): enhance WSS connectivity checks and update configuration * feat(autotls): add autotls check in config.yaml * feat(config): add WSS configuration for local setup * fix(autotls): ensure context cancellation is handled correctly during WSS connection tests * fix(autotls): update WSS connectivity test to disconnect everything before connecting * fix(autotls): add WSS group option and enhance error handling for WSS connectivity * feat(autotls): add certificate renewal testing options and enhance configuration * refactor(autotls): simplify certificate renewal options and update related configurations * fix(autotls): increase certificate renewal wait time to 500 seconds for improved testing * fix(autotls): extend timeout for certificate renewal to 15 minutes * feat(autotls): add ultralight group support and connectivity testing * feat(config): update local.yaml for ultralight configuration and enhance autotls checks * fix(autotls): re-enable certificate renewal test * feat(config): add local-dns-autotls and local-light-autotls configura… (#562) * feat(config): add local-dns-autotls and local-light-autotls configurations ons. * chore(config): comment out ultralight configuration * fix(config): update autotls configs * fix(autotls): simplify config * fix: use test cert from constant * fix(autotls): handle context cancellation during sleep intervals in c… (#566) * fix(autotls): handle context cancellation during sleep intervals in checks * feat(autotls): add UnderlayPollInterval option and enhance WSS underlay checks * refactor(config): reorder struct fields for clarity in check options * refactor(autotls): simplify options structure and improve WSS underlay checks * feat(config): add light node configuration for autotls support in local.yaml * feat(autotls): autotls forge validation (#569) * feat(autotls): add p2p-forge address validation and certificate renewal verification * chore: fix lint issue --------- Co-authored-by: Akrem Chabchoub <121046693+akrem-chabchoub@users.noreply.github.com> Co-authored-by: akrem-chabchoub <chabchoub.akrem@gmail.com>
akrem-chabchoub
added a commit
that referenced
this pull request
Mar 31, 2026
#566) * fix(autotls): handle context cancellation during sleep intervals in checks * feat(autotls): add UnderlayPollInterval option and enhance WSS underlay checks * refactor(config): reorder struct fields for clarity in check options * refactor(autotls): simplify options structure and improve WSS underlay checks
akrem-chabchoub
added a commit
that referenced
this pull request
Mar 31, 2026
#572) * feat: add new flags * fix(k8s): ensure p2p-wss service port is created when configured * chore: add comments to local.yaml * fix(config): update autotls-registration-endpoint * feat(autotls): add autotls support with pebble and p2p-forge (#559) * feat(autotls): add auto tls support in init containers and configuration options * fix(k8s): fix EnvVar handling by adding hasValues method and improving toK8S conversion * feat(config): add p2p-wss support and update autotls registration endpoint in configuration files * chore(config): remove unneeded p2p-wss settings * feat(autotls): implement autotls check functionality (#560) * feat(autotls): implement autotls check functionality * feat(autotls): enhance WSS connectivity checks and update configuration * feat(autotls): add autotls check in config.yaml * feat(config): add WSS configuration for local setup * fix(autotls): ensure context cancellation is handled correctly during WSS connection tests * fix(autotls): update WSS connectivity test to disconnect everything before connecting * fix(autotls): add WSS group option and enhance error handling for WSS connectivity * feat(autotls): add certificate renewal testing options and enhance configuration * refactor(autotls): simplify certificate renewal options and update related configurations * fix(autotls): increase certificate renewal wait time to 500 seconds for improved testing * fix(autotls): extend timeout for certificate renewal to 15 minutes * feat(autotls): add ultralight group support and connectivity testing * feat(config): update local.yaml for ultralight configuration and enhance autotls checks * fix(autotls): re-enable certificate renewal test * feat(config): add local-dns-autotls and local-light-autotls configura… (#562) * feat(config): add local-dns-autotls and local-light-autotls configurations ons. * chore(config): comment out ultralight configuration * fix(config): update autotls configs * fix(autotls): simplify config * fix: use test cert from constant * fix(autotls): handle context cancellation during sleep intervals in c… (#566) * fix(autotls): handle context cancellation during sleep intervals in checks * feat(autotls): add UnderlayPollInterval option and enhance WSS underlay checks * refactor(config): reorder struct fields for clarity in check options * refactor(autotls): simplify options structure and improve WSS underlay checks * feat(config): add light node configuration for autotls support in local.yaml * fix: git conflicts * chore: fix lint issue * feat(autotls): add support for custom DNS resolver and include Pebble CA certificate * lint forge test file * feat(autotls): enhance Pebble CA certificate fetching and direct DNS queries * chore: update forge DNS address in local configuration * chore: update forgeDNSAddr in check.go * chore: enhance DNS resolution handling with custom DNS server support * chore: remove unnecessary changes related to pebble * chore: update package imports * chore: rename ForgeDNSAddr * feat(autotls): add Forge TLS host address and Pebble management URL support * fix(autotls): increase certificate renewal wait time to accommodate expiry and check interval * fix(autotls): extend timeout for certificate renewal process * fix(autotls): adjust certificate renewal logic and improve logging for clarity * fix(autotls): refine certificate renewal process with enhanced retry logic and improved snapshot handling * refator: rm test forge * feat(autotls): introduce support for multiple AutoTLS groups and add P2P WSS node port configuration * refactor(autotls): consolidate HTTP client configuration for Pebble management API calls * fix: lint check.go * refactor(autotls): encapsulate Pebble management API interactions in a dedicated service * fix(autotls): enhance logging for certificate snapshot dialing errors * refactor(autotls): move certificate renewal logic to a dedicated function and improve structure * docs(autotls): add README for AutoTLS check detailing functionality, configuration, and package layout * docs(autotls): add sequence diagram to README for better visualization of AutoTLS check process * chore(config): update local.yaml with new addresses for forge DNS and Pebble management URL * chore(config): add local usage notes for forge DNS and Pebble management URL in local.yaml * fix: set timeout for insecure Pebble HTTP client * chore(config): update local.yaml with valid values for CI * chore(deps): update go.mod to include new dependencies and indirect packages --------- Co-authored-by: Ljubisa Gacevic <ljubisa.rs@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.