refactor: Separate Redis host and port configuration

This commit refactors the Redis connection settings to use distinct `REDIS_HOST` and `REDIS_PORT` environment variables instead of a single `REDIS_URL`. This change improves configuration clarity and flexibility across environments.

Additionally, it removes an incorrect custom `save` method from the `User` model. The previous implementation improperly set the `last_login` field upon user creation. This logic is flawed, as the `last_login` field should be managed by Django's authentication backend during the actual login process.

Author: RandomProgramm3r

.env.example

@@ -12,4 +12,5 @@ POSTGRES_PASSWORD=your_postgres_password
 POSTGRES_PORT=5432
 POSTGRES_USERNAME=your_postgres_username
 
-REDIS_URL=redis://redis:6379/0
+REDIS_HOST=redis
+REDIS_PORT=6379

promo_code/promo_code/settings.py

@@ -142,11 +142,13 @@ def load_bool(name, default):
         'PORT': os.getenv('POSTGRES_PORT', '5432'),
     },
 }
+REDIS_HOST = os.getenv('REDIS_HOST', 'redis')
+REDIS_PORT = os.getenv('REDIS_PORT', '6379')
 
 CACHES = {
     'default': {
         'BACKEND': 'django_redis.cache.RedisCache',
-        'LOCATION': os.getenv('REDIS_URL', 'redis://127.0.0.1:6379/0'),
+        'LOCATION': f'redis://{REDIS_HOST}:{REDIS_PORT}/0',
         'OPTIONS': {
             'CLIENT_CLASS': 'django_redis.client.DefaultClient',
         },

promo_code/user/models.py

@@ -78,12 +78,6 @@ class User(
     def __str__(self):
         return self.email
 
-    def save(self, *args, **kwargs):
-        if not self.pk:
-            self.last_login = django.utils.timezone.now()
-
-        super().save(*args, **kwargs)
-
 
 class PromoLike(django.db.models.Model):
     id = django.db.models.UUIDField(