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

chore(deps): implement github.com/qdm12/golibs/command locally #2418

Merged
merged 2 commits into from
Aug 21, 2024

Conversation

qdm12
Copy link
Owner

@qdm12 qdm12 commented Aug 16, 2024

  • Local implementation
  • Better implementation (no closing of channels, simpler comments, *exec.Cmd arguments while maintaining testability)
  • Line buffer can increase above the previous maximum of 64KB up to 20MB if needed - allows to debug Bug: protonvpn openvpn disconnects occasionally #2407
  • Test run it

@qdm12 qdm12 added the Status: 🔴 Blocked Blocked by another issue or pull request label Aug 16, 2024
Base automatically changed from dnsv2 to master August 21, 2024 12:35
qdm12 added 2 commits August 21, 2024 12:41
- Use `exec.Cmd` as argument instead of `ExecCmd` interface
- Return read only channels from the Start method
- Not closing stdout and stderr channels anymore since there is no need
@qdm12 qdm12 force-pushed the drop-golibs-command branch from 7770935 to 20cb894 Compare August 21, 2024 12:41
@qdm12 qdm12 marked this pull request as ready for review August 21, 2024 13:21
@qdm12 qdm12 merged commit a2b3d7e into master Aug 21, 2024
5 checks passed
@qdm12 qdm12 deleted the drop-golibs-command branch August 21, 2024 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: 🔴 Blocked Blocked by another issue or pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant