Merge pull request 'fix: add nil check in Forwarder.Close to avoid panic (issue #36 )' (#37 ) from staging into main

Reviewed-on: #37
fix: add nil check in Forwarder.Close to avoid panic (issue #36 )
2025-12-12 08:36:51 +00:00 · 2025-12-12 15:36:07 +07:00 · 2025-12-09 16:40:44 +00:00 · 2025-12-08 16:03:33 +00:00 · 2025-12-08 15:47:01 +00:00 · 2025-12-08 14:13:23 +00:00
1 changed files with 1 additions and 1 deletions
--- a/session/forwarder/forwarder.go
+++ b/session/forwarder/forwarder.go
@ -171,7 +171,7 @@ func (f *Forwarder) WriteBadGatewayResponse(dst io.Writer) {
 }
 func (f *Forwarder) Close() error {
-	if f.GetTunnelType() != types.HTTP {
+	if f.GetListener() != nil {
 		return f.Listener.Close()
 	}
 	return nil
Author	SHA1	Message	Date
bagas	5c6826fe89	Merge pull request 'fix: add nil check in Forwarder.Close to avoid panic (issue #36 )' (#37 ) from staging into main All checks were successful Docker Build and Push / build-and-push (push) Successful in 4m9s Details Reviewed-on: #37	2025-12-12 08:36:51 +00:00
bagas	2725975d82	fix: add nil check in Forwarder.Close to avoid panic (issue #36 ) All checks were successful Docker Build and Push / build-and-push (push) Successful in 4m8s Details	2025-12-12 15:36:07 +07:00
bagas	b484981017	Merge pull request 'staging' (#35 ) from staging into main All checks were successful Docker Build and Push / build-and-push (push) Successful in 4m22s Details Reviewed-on: #35	2025-12-09 16:40:44 +00:00
bagas	aceecfd14c	Merge pull request 'Update LICENSE.md' (#34 ) from staging into main All checks were successful Docker Build and Push / build-and-push (push) Successful in 6m44s Details Reviewed-on: #34	2025-12-08 16:03:33 +00:00
bagas	1de7155771	Merge pull request 'fix: prevent race condition when channel is used before being set' (#33 ) from staging into main All checks were successful Docker Build and Push / build-and-push (push) Successful in 3m46s Details Reviewed-on: #33	2025-12-08 15:47:01 +00:00
bagas	79d77497a0	Merge pull request 'update: readme' (#32 ) from staging into main All checks were successful Docker Build and Push / build-and-push (push) Successful in 6m22s Details Reviewed-on: #32	2025-12-08 14:13:23 +00:00
bagas	20b90c1727	Merge pull request 'Update .gitea/workflows/build.yml' (#31 ) from staging into main All checks were successful Docker Build and Push / build-and-push (push) Successful in 5m9s Details Reviewed-on: #31	2025-12-08 13:37:07 +00:00
bagas	a1e920f6b5	Merge pull request 'refactor: remove magic numbers and replace with named constants' (#30 ) from staging into main All checks were successful Docker Build and Push / build-and-push (push) Successful in 6m10s Details Reviewed-on: bagas/tunnl_please#30	2025-12-08 12:46:56 +00:00
bagas	b5045409cb	Merge pull request 'feat: add droping conn command' (#29 ) from staging into main All checks were successful Docker Build and Push / build-and-push (push) Successful in 4m30s Details Reviewed-on: bagas/tunnl_please#29	2025-12-07 09:30:47 +00:00
bagas	33e6ad08d7	Merge pull request 'staging' (#28 ) from staging into main All checks were successful Docker Build and Push / build-and-push (push) Successful in 6m0s Details Reviewed-on: bagas/tunnl_please#28	2025-12-06 16:47:20 +00:00
bagas	fd513d7bc9	Merge pull request 'refactor: instantiate new session object once forwarding is approved' (#27 ) from staging into main All checks were successful Docker Build and Push / build-and-push (push) Successful in 6m2s Details Reviewed-on: bagas/tunnl_please#27	2025-12-06 16:14:22 +00:00
bagas	73e7df6a3b	Merge pull request 'fix: correct read/write handling in CustomWriter' (#26 ) from staging into main All checks were successful Docker Build and Push / build-and-push (push) Successful in 5m14s Details Reviewed-on: bagas/tunnl_please#26	2025-12-06 15:20:11 +00:00
bagas	d2c5b2a4db	Merge pull request 'staging' (#25 ) from staging into main All checks were successful Docker Build and Push / build-and-push (push) Successful in 4m21s Details Reviewed-on: bagas/tunnl_please#25	2025-12-05 17:09:59 +00:00
bagas	cad22cd25a	Merge pull request 'refactor: change to gitea action' (#21 ) from staging into main All checks were successful Docker Build and Push / build-and-push (push) Successful in 4m19s Details Reviewed-on: bagas/tunnl_please#21	2025-12-03 14:21:50 +00:00
bagas	0cb02f5220	Merge pull request 'refactor: remove profiling' (#20 ) from staging into main Some checks failed Docker Build and Push / build-and-push (push) Has been cancelled Details Reviewed-on: bagas/tunnl_please#20	2025-12-03 14:20:52 +00:00
bagas	7bee2f2c9b	Merge pull request 'staging' (#19 ) from staging into main Some checks failed Docker Build and Push / build-and-push (push) Failing after 2s Details Reviewed-on: bagas/tunnl_please#19	2025-12-03 14:18:20 +00:00
bagas	54069ad305	Merge pull request 'fix: http forwarding stuck' (#9 ) from staging into main Some checks failed Docker Build and Push / build-and-push (push) Failing after 3s Details Reviewed-on: bagas/tunnl_please#9	2025-11-28 12:36:00 +00:00
bagas	5bf618aa32	Merge pull request 'fix: logic error in port checking' (#8 ) from staging into main Some checks failed Docker Build and Push / build-and-push (push) Has been cancelled Details Reviewed-on: bagas/tunnl_please#8	2025-11-28 10:34:28 +00:00
bagas	34041a9fe6	Merge pull request 'refactor: handle error' (#7 ) from staging into main Some checks failed Docker Build and Push / build-and-push (push) Has been cancelled Details Reviewed-on: bagas/tunnl_please#7	2025-11-28 09:57:47 +00:00
bagas	c6d64aff3a	Merge pull request 'refactor: separate session responsibilities and inject dependencies' (#6 ) from staging into main Some checks failed Docker Build and Push / build-and-push (push) Has been cancelled Details Reviewed-on: bagas/tunnl_please#6	2025-11-28 06:30:09 +00:00
bagas	2e5a4e0b71	Merge pull request 'fix: close connection on TCP/IP request timeout' (#5 ) from staging into main Some checks failed Docker Build and Push / build-and-push (push) Has been cancelled Details Reviewed-on: bagas/tunnl_please#5	2025-11-27 16:04:55 +00:00