fix networking dropping clients after some requests
This commit is contained in:
@@ -1,5 +1,49 @@
|
|||||||
extends Node
|
extends Node
|
||||||
|
|
||||||
|
static var gurt_clients: Dictionary = {}
|
||||||
|
static var client_last_used: Dictionary = {}
|
||||||
|
static var client_timeout_ms: int = 30000
|
||||||
|
|
||||||
|
func _ready():
|
||||||
|
var timer = Timer.new()
|
||||||
|
timer.wait_time = 10.0
|
||||||
|
timer.autostart = true
|
||||||
|
timer.timeout.connect(_cleanup_idle_clients)
|
||||||
|
add_child(timer)
|
||||||
|
|
||||||
|
func _cleanup_idle_clients():
|
||||||
|
var current_time = Time.get_ticks_msec()
|
||||||
|
var to_remove = []
|
||||||
|
|
||||||
|
for domain in client_last_used:
|
||||||
|
if current_time - client_last_used[domain] > client_timeout_ms:
|
||||||
|
to_remove.append(domain)
|
||||||
|
|
||||||
|
for domain in to_remove:
|
||||||
|
if gurt_clients.has(domain):
|
||||||
|
gurt_clients[domain].disconnect()
|
||||||
|
gurt_clients.erase(domain)
|
||||||
|
client_last_used.erase(domain)
|
||||||
|
|
||||||
|
func get_or_create_gurt_client(domain: String) -> GurtProtocolClient:
|
||||||
|
_cleanup_idle_clients()
|
||||||
|
|
||||||
|
if gurt_clients.has(domain):
|
||||||
|
client_last_used[domain] = Time.get_ticks_msec()
|
||||||
|
return gurt_clients[domain]
|
||||||
|
|
||||||
|
var client = GurtProtocolClient.new()
|
||||||
|
|
||||||
|
for ca_cert in CertificateManager.trusted_ca_certificates:
|
||||||
|
client.add_ca_certificate(ca_cert)
|
||||||
|
|
||||||
|
if not client.create_client_with_dns(30, GurtProtocol.DNS_SERVER_IP, GurtProtocol.DNS_SERVER_PORT):
|
||||||
|
return null
|
||||||
|
|
||||||
|
gurt_clients[domain] = client
|
||||||
|
client_last_used[domain] = Time.get_ticks_msec()
|
||||||
|
return client
|
||||||
|
|
||||||
func fetch_image(url: String) -> ImageTexture:
|
func fetch_image(url: String) -> ImageTexture:
|
||||||
if url.is_empty():
|
if url.is_empty():
|
||||||
return null
|
return null
|
||||||
@@ -170,15 +214,6 @@ func fetch_gurt_resource(url: String, as_binary: bool = false):
|
|||||||
|
|
||||||
var network_request = NetworkManager.start_request(gurt_url, "GET", false)
|
var network_request = NetworkManager.start_request(gurt_url, "GET", false)
|
||||||
|
|
||||||
var client = GurtProtocolClient.new()
|
|
||||||
|
|
||||||
for ca_cert in CertificateManager.trusted_ca_certificates:
|
|
||||||
client.add_ca_certificate(ca_cert)
|
|
||||||
|
|
||||||
if not client.create_client_with_dns(30, GurtProtocol.DNS_SERVER_IP, GurtProtocol.DNS_SERVER_PORT):
|
|
||||||
NetworkManager.fail_request(network_request.id, "Failed to create GURT client")
|
|
||||||
return ""
|
|
||||||
|
|
||||||
var host_domain = gurt_url
|
var host_domain = gurt_url
|
||||||
if host_domain.begins_with("gurt://"):
|
if host_domain.begins_with("gurt://"):
|
||||||
host_domain = host_domain.substr(7)
|
host_domain = host_domain.substr(7)
|
||||||
@@ -186,11 +221,15 @@ func fetch_gurt_resource(url: String, as_binary: bool = false):
|
|||||||
if slash_pos != -1:
|
if slash_pos != -1:
|
||||||
host_domain = host_domain.substr(0, slash_pos)
|
host_domain = host_domain.substr(0, slash_pos)
|
||||||
|
|
||||||
|
var client = get_or_create_gurt_client(host_domain)
|
||||||
|
if client == null:
|
||||||
|
NetworkManager.fail_request(network_request.id, "Failed to create GURT client")
|
||||||
|
return PackedByteArray() if as_binary else ""
|
||||||
|
|
||||||
var response = client.request(gurt_url, {
|
var response = client.request(gurt_url, {
|
||||||
"method": "GET",
|
"method": "GET",
|
||||||
"headers": {"Host": host_domain}
|
"headers": {"Host": host_domain}
|
||||||
})
|
})
|
||||||
client.disconnect()
|
|
||||||
|
|
||||||
if not response or not response.is_success:
|
if not response or not response.is_success:
|
||||||
var error_msg = "Failed to load GURT resource"
|
var error_msg = "Failed to load GURT resource"
|
||||||
|
|||||||
@@ -1,6 +1,10 @@
|
|||||||
class_name Main
|
class_name Main
|
||||||
extends Control
|
extends Control
|
||||||
|
|
||||||
|
static var gurt_clients: Dictionary = {}
|
||||||
|
static var client_last_used: Dictionary = {}
|
||||||
|
static var client_timeout_ms: int = 30000
|
||||||
|
|
||||||
@onready var website_container: Control = %WebsiteContainer
|
@onready var website_container: Control = %WebsiteContainer
|
||||||
@onready var tab_container: TabManager = $VBoxContainer/TabContainer
|
@onready var tab_container: TabManager = $VBoxContainer/TabContainer
|
||||||
@onready var search_bar: LineEdit = $VBoxContainer/HBoxContainer/LineEdit
|
@onready var search_bar: LineEdit = $VBoxContainer/HBoxContainer/LineEdit
|
||||||
@@ -75,6 +79,39 @@ func _ready():
|
|||||||
call_deferred("update_navigation_buttons")
|
call_deferred("update_navigation_buttons")
|
||||||
call_deferred("_handle_startup_behavior")
|
call_deferred("_handle_startup_behavior")
|
||||||
|
|
||||||
|
func _cleanup_idle_clients():
|
||||||
|
var current_time = Time.get_ticks_msec()
|
||||||
|
var to_remove = []
|
||||||
|
|
||||||
|
for domain in client_last_used:
|
||||||
|
if current_time - client_last_used[domain] > client_timeout_ms:
|
||||||
|
to_remove.append(domain)
|
||||||
|
|
||||||
|
for domain in to_remove:
|
||||||
|
if gurt_clients.has(domain):
|
||||||
|
gurt_clients[domain].disconnect()
|
||||||
|
gurt_clients.erase(domain)
|
||||||
|
client_last_used.erase(domain)
|
||||||
|
|
||||||
|
func get_or_create_gurt_client(domain: String) -> GurtProtocolClient:
|
||||||
|
_cleanup_idle_clients()
|
||||||
|
|
||||||
|
if gurt_clients.has(domain):
|
||||||
|
client_last_used[domain] = Time.get_ticks_msec()
|
||||||
|
return gurt_clients[domain]
|
||||||
|
|
||||||
|
var client = GurtProtocolClient.new()
|
||||||
|
|
||||||
|
for ca_cert in CertificateManager.trusted_ca_certificates:
|
||||||
|
client.add_ca_certificate(ca_cert)
|
||||||
|
|
||||||
|
if not client.create_client_with_dns(30, GurtProtocol.DNS_SERVER_IP, GurtProtocol.DNS_SERVER_PORT):
|
||||||
|
return null
|
||||||
|
|
||||||
|
gurt_clients[domain] = client
|
||||||
|
client_last_used[domain] = Time.get_ticks_msec()
|
||||||
|
return client
|
||||||
|
|
||||||
func _input(_event: InputEvent) -> void:
|
func _input(_event: InputEvent) -> void:
|
||||||
if Input.is_action_just_pressed("DevTools"):
|
if Input.is_action_just_pressed("DevTools"):
|
||||||
_toggle_dev_tools()
|
_toggle_dev_tools()
|
||||||
@@ -151,19 +188,21 @@ func fetch_gurt_content_async(gurt_url: String, tab: Tab, original_url: String,
|
|||||||
|
|
||||||
func _perform_gurt_request_threaded(request_data: Dictionary) -> Dictionary:
|
func _perform_gurt_request_threaded(request_data: Dictionary) -> Dictionary:
|
||||||
var gurt_url: String = request_data.gurt_url
|
var gurt_url: String = request_data.gurt_url
|
||||||
var client = GurtProtocolClient.new()
|
|
||||||
|
|
||||||
for ca_cert in CertificateManager.trusted_ca_certificates:
|
var host_domain = gurt_url
|
||||||
client.add_ca_certificate(ca_cert)
|
if host_domain.begins_with("gurt://"):
|
||||||
|
host_domain = host_domain.substr(7)
|
||||||
|
var slash_pos = host_domain.find("/")
|
||||||
|
if slash_pos != -1:
|
||||||
|
host_domain = host_domain.substr(0, slash_pos)
|
||||||
|
|
||||||
if not client.create_client_with_dns(30, GurtProtocol.DNS_SERVER_IP, GurtProtocol.DNS_SERVER_PORT):
|
var client = get_or_create_gurt_client(host_domain)
|
||||||
client.disconnect()
|
if client == null:
|
||||||
return {"success": false, "error": "Failed to connect to GURT DNS server at " + GurtProtocol.DNS_SERVER_IP + ":" + str(GurtProtocol.DNS_SERVER_PORT)}
|
return {"success": false, "error": "Failed to connect to GURT DNS server at " + GurtProtocol.DNS_SERVER_IP + ":" + str(GurtProtocol.DNS_SERVER_PORT)}
|
||||||
|
|
||||||
var response = client.request(gurt_url, {
|
var response = client.request(gurt_url, {
|
||||||
"method": "GET"
|
"method": "GET"
|
||||||
})
|
})
|
||||||
client.disconnect()
|
|
||||||
|
|
||||||
if not response or not response.is_success:
|
if not response or not response.is_success:
|
||||||
var error_msg = "Connection failed"
|
var error_msg = "Connection failed"
|
||||||
|
|||||||
Reference in New Issue
Block a user