Skip to content

Explicitly require `ipaddress` gem

Mehmet Emin INAC requested to merge require_ipaddress into master

What does this MR do?

ipaddress gem is not loaded in the development environment by default which is causing an error in IpAddressValidator because it's depending on a constant defined in that gem. We should explicitly require the gem to solve this issue.

Note: I believe we should also explicitly add this gem into the Gemfile instead of depending on other gems using it.

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team

Merge request reports