ecometrica / django-multisite

Multisite in django — use one Django app to serve multiple domains
BSD 3-Clause "New" or "Revised" License
138 stars 42 forks source link

Remove deprecated warning with django 3 #73

Open kutera opened 4 years ago

kutera commented 4 years ago

Hello, On Django 3 a warning appears. Here a small commit with correction to avoid it.

I noticed a regression with tldextract==3.0.2 :
_middelware.py 245: tldextract.TLDExtract(cache_file=self.pslcache) The cache_file option is replaced by cache_dir.

My work around was to add tldextract==2.2.3 in the requirements.

Best Regards

damon-rand commented 4 years ago

Thank you for the contribution @kutera