remove RedisDsn.user_required = True (#1658)

* remove RedisDsn.user_required = True

* update docs

changes/1275-samuelcolvin.md

@@ -0,0 +1 @@
+Remove `user_required = True` from `RedisDsn`, neither user nor password are required.

docs/usage/types.md

@@ -508,8 +508,9 @@ For URI/URL validation the following types are available:
 - `AnyUrl`: any scheme allowed, TLD not required
 - `AnyHttpUrl`: schema `http` or `https`, TLD not required
 - `HttpUrl`: schema `http` or `https`, TLD required, max length 2083
-- `PostgresDsn`: schema `postgres` or `postgresql`, userinfo required, TLD not required
-- `RedisDsn`: schema `redis`, userinfo required, tld not required
+- `PostgresDsn`: schema `postgres` or `postgresql`, user info required, TLD not required
+- `RedisDsn`: schema `redis`, user info not required, tld not required (CHANGED: user info not required from
+  **v1.6** onwards)
 - `stricturl`, method with the following keyword arguments:
     - `strip_whitespace: bool = True`
     - `min_length: int = 1`

pydantic/networks.py

@@ -274,7 +274,6 @@ class PostgresDsn(AnyUrl):
 
 class RedisDsn(AnyUrl):
     allowed_schemes = {'redis'}
-    user_required = True
 
 
 def stricturl(

tests/test_networks.py

@@ -324,10 +324,11 @@ class Model(BaseModel):
         Model(a='http://example.org')
     assert exc_info.value.errors()[0]['type'] == 'value_error.url.scheme'
 
-    with pytest.raises(ValidationError) as exc_info:
-        Model(a='redis://localhost:5432/app')
-    error = exc_info.value.errors()[0]
-    assert error == {'loc': ('a',), 'msg': 'userinfo required in URL but missing', 'type': 'value_error.url.userinfo'}
+    # password is not required for redis
+    m = Model(a='redis://localhost:5432/app')
+    assert m.a == 'redis://localhost:5432/app'
+    assert m.a.user is None
+    assert m.a.password is None
 
 
 def test_custom_schemes():