-
Notifications
You must be signed in to change notification settings - Fork 112
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 validation: validate IPv6 proxy URL only when IS_IPV6 is set to True #15413
Fix validation: validate IPv6 proxy URL only when IS_IPV6 is set to True #15413
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe default=''
might be helpful, but not necessary. ACK
@ogajduse I just backported Ipv6 to 6.15 and it likely won't go beyond that. If needed we can do cherry-pick later. |
Cherrypicking this PR should follow cherrypicking of previous validators fixes - #15277 |
…rue (SatelliteQE#15413) (cherry picked from commit 90ffb9b)
…rue (SatelliteQE#15413) (cherry picked from commit 90ffb9b)
… set to True (#15423) Fix validation: validate IPv6 proxy URL only when IS_IPV6 is set to True (#15413) (cherry picked from commit 90ffb9b) Co-authored-by: Ondřej Gajdušek <[email protected]>
Problem Statement
Using latest robottelo, including the IPv6-related changes, I am running into
This error is caused by a validator that enforces string type, but it is setting the value to None itself. :meow_shrugs:
Solution
Utilize conditional validation - validate the IPv6 proxy URL setting only when IS_IPV6 is set to True
https://www.dynaconf.com/validation/#conditional-validation
Related Issues
This should be fixed before #15289 goes in