Bug fix

Author: Simple-Tracker

README.en.md

@@ -147,7 +147,7 @@ Docker version is configured through the same name variable configuration, which
 | ignoreFailureExit | bool | false | Ignore failure exit. If enabled, it will continue to retry after first detection of the client fails or authentication fails |
 | sleepTime | uint32 | 20 (MicroSec) | Query waiting time of each Torrent Peers. Short interval can make blocking Peer faster but may cause client lag, Long interval can help average CPU usage |
 | timeout | uint32 | 6 (MillSec) | Request timeout. If interval is too short, peer may not be properly blocked. If interval is too long, timeout request will affect blocking other peer |
-| proxy | string | Auto | Use proxy. Proxy will still automatically detect proxy on first load. Empty: Disable proxy; Auto: Automatic (use proxy only for external resources); All: Use proxy |
+| proxy | string | Auto | Use proxy. Still automatically detect proxy on first load. Empty: Disable proxy; Auto: Automatic (use proxy only for external resources); All: Use proxy |
 | longConnection | bool | true | Long connection. Enable to reduce resource consumption |
 | logPath | string | logs | Log path. Must enable logToFile |
 | logToFile | bool | true | Log general information to file. If enabled, it can be used for general analysis and statistical purposes |

request.go

@@ -8,7 +8,7 @@ import (
 
 var fetchFailedCount = 0
 
-func NewRequest(isPOST bool, url string, postdata string, external bool, withHeader *map[string]string) *http.Request {
+func NewRequest(isPOST bool, url string, postdata string, clientReq bool, withHeader *map[string]string) *http.Request {
 	var request *http.Request
 	var err error
 
@@ -47,26 +47,28 @@ func NewRequest(isPOST bool, url string, postdata string, external bool, withHea
 		request.Header.Set("Content-Type", "application/x-www-form-urlencoded")
 	}
 
-	if currentClientType == "Transmission" && external && Tr_csrfToken != "" {
-		request.Header.Set("X-Transmission-Session-Id", Tr_csrfToken)
-	}
+	if clientReq {
+		if currentClientType == "Transmission" && Tr_csrfToken != "" {
+			request.Header.Set("X-Transmission-Session-Id", Tr_csrfToken)
+		}
 
-	if external && config.UseBasicAuth && config.ClientUsername != "" {
-		request.SetBasicAuth(config.ClientUsername, config.ClientPassword)
+		if config.UseBasicAuth && config.ClientUsername != "" {
+			request.SetBasicAuth(config.ClientUsername, config.ClientPassword)
+		}
 	}
 
 	return request
 }
-func Fetch(url string, tryLogin bool, external bool, withHeader *map[string]string) (int, http.Header, []byte) {
-	request := NewRequest(false, url, "", external, withHeader)
+func Fetch(url string, tryLogin bool, clientReq bool, withHeader *map[string]string) (int, http.Header, []byte) {
+	request := NewRequest(false, url, "", clientReq, withHeader)
 	if request == nil {
 		return -1, nil, nil
 	}
 
 	var response *http.Response
 	var err error
 
-	if external {
+	if clientReq {
 		response, err = httpClient.Do(request)
 	} else {
 		response, err = httpClientExternal.Do(request)
@@ -141,16 +143,16 @@ func Fetch(url string, tryLogin bool, external bool, withHeader *map[string]stri
 
 	return response.StatusCode, response.Header, responseBody
 }
-func Submit(url string, postdata string, tryLogin bool, external bool, withHeader *map[string]string) (int, http.Header, []byte) {
-	request := NewRequest(true, url, postdata, external, withHeader)
+func Submit(url string, postdata string, tryLogin bool, clientReq bool, withHeader *map[string]string) (int, http.Header, []byte) {
+	request := NewRequest(true, url, postdata, clientReq, withHeader)
 	if request == nil {
 		return -1, nil, nil
 	}
 
 	var response *http.Response
 	var err error
 
-	if external {
+	if clientReq {
 		response, err = httpClient.Do(request)
 	} else {
 		response, err = httpClientExternal.Do(request)