ManageIQ / more_core_extensions

MoreCoreExtensions are a set of core extensions beyond those provided by ActiveSupport.
MIT License
5 stars 23 forks source link

Allow for > 6 chars TLDs in email addresses #31

Closed imtayadeway closed 7 years ago

imtayadeway commented 7 years ago

As per RFC 1035

Addresses https://github.com/ManageIQ/manageiq/issues/12781

Thanks @Fryguy for the suggestion on this one. In the above issue you mentioned that we could also go further and use the regex outlined in http://www.regular-expressions.info/email.html. I do think that's a good idea and would like to do that, but am thinking I might do that in a follow up PR as it breaks some existing tests. I don't think that's a problem, but I'm being extra cautious - it should at least be easier to revert if it breaks anyone's workflow and keep this enhancement.

Also, TIL a new word: prestidigitation :grin:

@miq-bot add-label enhacement @miq-bot assign @Fryguy

miq-bot commented 7 years ago

@imtayadeway Cannot apply the following label because they are not recognized: enhacement

miq-bot commented 7 years ago

Checked commit https://github.com/imtayadeway/more_core_extensions/commit/79e089fa08a654dab7a6c045030e1433c8cb6610 with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1 2 files checked, 4 offenses detected

lib/more_core_extensions/core_ext/string/formats.rb

Fryguy commented 7 years ago

The Hakiri thing is independent of this PR, so I'm ignoring that.