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

fix CloseAllConns #104

Merged
merged 25 commits into from
Oct 28, 2023
Merged

fix CloseAllConns #104

merged 25 commits into from
Oct 28, 2023

Conversation

yunginnanet
Copy link
Owner

No description provided.

@@ -1,9 +1,9 @@
name: tests

Choose a reason for hiding this comment

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

GPT summary of ab45a3 - ac5409:

  • Changed the syntax of branches from branches: ["main", "development"] to branches: [ "main", "development" ] in the push and pull_request sections.

@@ -1,9 +1,9 @@
package prox5

Choose a reason for hiding this comment

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

GPT summary of b09470 - 8e00fb:

  • Changed context import to time
  • Modified CloseAllConns to (maybe) close all connections in progress
  • Added a quit method to Close which kills the connections

@@ -100,10 +100,7 @@ func (p5 *ProxyEngine) jobSpawner() {
// case <-p5.ctx.Done():

Choose a reason for hiding this comment

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

GPT summary of 8c64fd - a25fe3:

  • Removed code related to locking and unlocking the pending list
  • Added a call to the recycling function when the pending list is empty

@@ -78,10 +78,8 @@ type ProxyEngine struct {

Choose a reason for hiding this comment

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

GPT summary of bf8291 - 579706:

  • Removed conCtx and killConns fields from ProxyEngine struct
  • Added ctx and quit fields to ProxyEngine struct
  • Added conKiller field to ProxyEngine struct
  • Removed context.WithCancel call for conCtx and killConns fields

@@ -1,3 +1,5 @@
git.tcp.direct/kayos/common v0.8.6 h1:lt8nv+PrgAcbiOnbKUt7diza5hifR5fV3un6uIp/YVc=

Choose a reason for hiding this comment

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

GPT summary of fb1323 - f927ee:

  • Added two entries for version v0.8.6 of git.tcp.direct/kayos/common
  • Updated two entries for version v0.9.2 of git.tcp.direct/kayos/common
  • Added one entry for version v0.3.0 of git.tcp.direct/kayos/go-socks5

@@ -3,6 +3,7 @@ package prox5
import (

Choose a reason for hiding this comment

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

GPT summary of 80e4c3 - e82706:

  • Added import "io"
  • Added select statement to check for context errors
  • Added case <-p5.conKiller to select statement
  • Added case <-p5.conKiller to select statement within loop
  • Added _ = conn.Close() to select statement within loop

@@ -244,6 +244,7 @@ testLoop:
t.Fatal("no successful requests")

Choose a reason for hiding this comment

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

GPT summary of 411c72 - f64630:

  • Added a line to close all connections
  • Added a log statement to print the total successful requests

@github-actions
Copy link

GPT summary of d8f03bc:

  • Added a list.RUnlock() after the list.RLock() in the GetAnySOCKS() function dispense.go
  • Added a list.RUnlock() after checking if the proxy is still good dispense.go
  • Added a recycling() function call if the list is empty dispense.go
  • Lowered the sleep time from 15 to 50 milliseconds dispense.go

@github-actions
Copy link

GPT summary of 69187d7:

  • Removed the tpls channel and recycleTuples function daemons.go
  • Replaced the tpls channel with direct calls to Pending.add() daemons.go
  • Changed the iteration order for the proxyMap.plot depending on the GetRecyclerShuffleStatus() daemons.go

@github-actions
Copy link

GPT summary of 54e641f:

  • Removed the redu line from the recycling() function daemons.go

@github-actions
Copy link

GPT summary of 121543b:

  • Changed the logic for recycling SOCKS proxies dispense.go
  • Updated the logic for checking if a proxy is still good dispense.go
  • Removed unnecessary locks from the code dispense.go

@github-actions
Copy link

GPT summary of 5032231:

  • Removed an unnecessary error log dispense.go
  • Added a return false statement to the stillGood function dispense.go

@github-actions
Copy link

GPT summary of 2e85c81:

Error: couldn't generate summary

@github-actions
Copy link

GPT summary of 8497792:

Not generating summary for merge commits

@github-actions
Copy link

GPT summary of 39bc5ac:

@github-actions
Copy link

GPT summary of bbbc9b7:

@github-actions
Copy link

GPT summary of 05b9e36:

  • Lowered the time before a request is made from 500 to 300 milliseconds debug.go, prox5_test.go
  • Raised the amount of workers from 1 to 10 prox5_test.go
  • Raised the maximum amount of workers from 1 to 100 prox5_test.go
  • Raised the burst size of the bad proxy rate limiter from 5 to 10 proxy.go

@github-actions
Copy link

GPT summary of 7ae3ebd:

  • Changed the branches monitored for pushes from master and dev to master and development go.yml

@github-actions
Copy link

GPT summary of 40ed287:

  • Changed the t.Fatal to t.Error in the TestProx5 function prox5_test.go

@github-actions
Copy link

GPT summary of 4f3c842:

@github-actions
Copy link

GPT summary of d8c0a3a:

@github-actions
Copy link

GPT summary of 0c61142:

  • Removed github.com/ysmood/gotrace from go.mod and go.sum
  • Updated github.com/andybalholm/brotli from v1.0.4 to v1.0.5 in go.mod and go.sum go.mod, go.sum

@github-actions
Copy link

GPT summary of db3eaf3:

  • Changed branch names for push and pull requests from master to main go.yml

@github-actions
Copy link

GPT summary of a1c31c9:

Not generating summary for merge commits

@github-actions
Copy link

GPT summary of 3c6bba9:

Not generating summary for merge commits

@github-actions
Copy link

GPT summary of aded278:

  • Changed the GetTotalValidated method to add all valid proxy counts instead of individual addition stats.go

@codecov
Copy link

codecov bot commented Oct 28, 2023

Codecov Report

Merging #104 (e49afae) into main (a892a21) will increase coverage by 3.86%.
The diff coverage is 67.85%.

@@            Coverage Diff             @@
##             main     #104      +/-   ##
==========================================
+ Coverage   54.10%   57.96%   +3.86%     
==========================================
  Files          18       18              
  Lines        1303     1318      +15     
==========================================
+ Hits          705      764      +59     
+ Misses        541      504      -37     
+ Partials       57       50       -7     
Files Coverage Δ
daemons.go 83.13% <100.00%> (+23.83%) ⬆️
defs.go 89.01% <100.00%> (+3.29%) ⬆️
conductor.go 77.27% <82.35%> (+14.77%) ⬆️
mystery_dialer.go 48.71% <33.33%> (+0.97%) ⬆️

... and 5 files with indirect coverage changes

@github-actions
Copy link

GPT summary of 8592c4f:

  • Added environment variable to set the amount of tests prox5_test.go
  • Added os.Setenv call to set PROX5_SCALER_DEBUG to 1 prox5_test.go
  • Added a sync.Once to check for the expected amount of validated proxies prox5_test.go
  • Added os.Getenv call to get the amount of tests from environment variable prox5_test.go
  • Added entropy.GetOptimizedRand call to scale the amount of failures prox5_test.go
  • Added atomic.AddInt64 call to increase the failures variable prox5_test.go
  • Added entropy.RandSleepMS call to randomize the sleep duration prox5_test.go
  • Added os.Getenv call to get the duration of tests from environment variable prox5_test.go

@yunginnanet yunginnanet merged commit 2aa4580 into main Oct 28, 2023
5 checks passed
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.

None yet

1 participant