Why do you do this either way, ```options['login_form']``` is ```true``` or ```false```?
Shouldn't this depend on ```icon```, instead?
Something like:
```ruby
if icon
  Gitlab.config.omniauth.form_providers << provider
end
```