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: session resource warning #4

Closed

Conversation

cihanzereyalp
Copy link
Contributor

No description provided.

@fatihmete
Copy link
Owner

Merhaba,
Hatayı hangi koşullarda aldığınızı ve çözümün gerekçesini ekleyebilir misiniz?

@cihanzereyalp
Copy link
Contributor Author

Selamlar,
Hata yok sadece Unittestler sirasinda ResourceWarning olarak cikiyor. Açıkça kaynakların duzgun şekilde serbest bırakılması için session.close() şeklinde de kullanılabilir. Bir de self.httpsVerify için yazım yanlışı düzeltmesi içeriyor.

@fatihmete
Copy link
Owner

urllib3 kullanılarak session için bağlantının sürekli açık kalması sağlanıyormuş *. Belirttiğiniz gibi bağlantının sürekli açık kalması uyarıya neden oluyor.

PR8 session yapısını değiştirdiği için aşağıdaki şekilde her istek gerçekleştikten sonra self.session.close()çalıştırılması daha uygun olabilir.

...
def __make_request(self, url, params={}):
        params = self.__param_generator(params)
        request = self.session.get(url + params)
        self.session.close()
...

Değişiklikten sonra urllib3'ün loglarında her istek için yeni connection oluşturulduğu görülüyor:


DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): evds2.tcmb.gov.tr:443

@fatihmete
Copy link
Owner

v0.3.0 sürümüne ekledi.

@fatihmete fatihmete closed this Jul 9, 2023
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.

2 participants