jhao104/proxy_pool

Pattern Findings

57 matches across 4 categories. Click a row to expand file-level details.

Hyper-Verbose Identifiers28 hits · 32 pts

Severity	File	Line	Snippet
LOW	tests/unit/test_db_client.py	36	def test_redis_uri_no_password(self):
LOW	tests/unit/test_db_client.py	44	def test_ssdb_uri_no_password(self):
LOW	tests/unit/test_db_client.py	50	def test_unknown_db_type_raises(self):
LOW	tests/unit/test_config.py	63	def test_verify_timeout_default(self, conf):
LOW	tests/unit/test_config.py	66	def test_max_fail_count_default(self, conf):
LOW	tests/unit/test_config.py	69	def test_pool_size_min_default(self, conf):
LOW	tests/unit/test_config.py	92	def test_verify_timeout_override(self):
LOW	tests/unit/test_config.py	97	def test_max_fail_count_override(self):
LOW	tests/unit/test_validator.py	48	def test_invalid_proxy_format(self, proxy):
LOW	tests/unit/test_validator.py	67	def test_invalid_returns_false(self, proxy):
LOW	tests/unit/test_proxy.py	81	def test_to_json_is_valid_json(self):
LOW	tests/unit/test_proxy.py	87	def test_create_from_json_roundtrip(self):
LOW	tests/unit/test_proxy.py	98	def test_create_from_json_minimal(self):
LOW	tests/unit/test_proxy.py	106	def test_create_from_json_with_slash_source(self):
LOW	tests/unit/test_proxy.py	112	def test_to_dict_to_json_consistency(self):
LOW	tests/unit/test_proxy.py	169	def test_add_source_empty_string(self):
LOW	tests/integration/test_redis_client.py	57	def test_get_https_excludes_http(self, redis_client):
LOW	tests/integration/test_redis_client.py	63	def test_get_empty_returns_none(self, redis_client):
LOW	tests/integration/test_redis_client.py	97	def test_pop_empty_returns_none(self, redis_client):
LOW	tests/integration/test_redis_client.py	110	def test_get_all_https_filter(self, redis_client):
LOW	tests/integration/test_redis_client.py	144	def test_change_table_isolation(self, redis_client):
LOW	tests/integration/test_ssdb_client.py	56	def test_get_https_excludes_http(self, ssdb_client):
LOW	tests/integration/test_ssdb_client.py	62	def test_get_empty_returns_none(self, ssdb_client):
LOW	tests/integration/test_ssdb_client.py	96	def test_pop_empty_returns_none(self, ssdb_client):
LOW	tests/integration/test_ssdb_client.py	109	def test_get_all_https_filter(self, ssdb_client):
LOW	tests/integration/test_ssdb_client.py	143	def test_change_table_isolation(self, ssdb_client):
LOW	tests/api/test_proxy_api.py	27	def test_index_returns_api_list(self, client):
LOW	tests/api/test_proxy_api.py	118	def test_delete_calls_handler(self, client, mocks):

Excessive Try-Catch Wrapping15 hits · 16 pts

Severity	File	Line	Snippet
LOW	README.md	157	except Exception:
LOW	util/webRequest.py	79	except Exception as e:
LOW	util/webRequest.py	106	except Exception as e:
LOW	util/webRequest.py	131	except Exception as e:
MEDIUM	util/webRequest.py	128	def json(self):
LOW	fetcher/proxyFetcher.py	184	except Exception as e:
LOW	fetcher/proxyFetcher.py	273	except Exception as e:
LOW	fetcher/proxyFetcher.py	318	except Exception as e:
LOW	fetcher/proxyFetcher.py	333	except Exception as e:
LOW	docs/api.md	48	except Exception:
LOW	docs/extending/validator.md	49	except Exception as e:
LOW	docs/extending/validator.md	63	except Exception as e:
LOW	helper/fetch.py	46	except Exception as e:
LOW	helper/validator.py	67	except Exception as e:
LOW	helper/validator.py	79	except Exception as e:

Unused Imports10 hits · 10 pts

Severity	File	Line
LOW	util/six.py	28
LOW	util/six.py	30
LOW	util/six.py	41
LOW	util/six.py	41
LOW	util/six.py	43
LOW	util/six.py	43
LOW	tests/unit/test_validator.py	15
LOW	tests/unit/test_proxy.py	16
LOW	tests/integration/test_redis_client.py	17
LOW	tests/integration/test_redis_client.py	18

Over-Commented Block4 hits · 4 pts

Severity	File	Line	Snippet
LOW	setting.py	81	POOL_SIZE_MIN = 20
LOW	fetcher/proxyFetcher.py	341	# from requests import Session
LOW	fetcher/proxyFetcher.py	361	# """
LOW	fetcher/proxyFetcher.py	381	# import base64

Score History

Severity Breakdown

Pattern Findings