Skip to content
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

Inviting users is not possible with simple SMTP settings! #2002

Closed
fduraibi opened this issue Jan 21, 2016 · 2 comments
Closed

Inviting users is not possible with simple SMTP settings! #2002

fduraibi opened this issue Jan 21, 2016 · 2 comments

Comments

@fduraibi
Copy link
Contributor

This issue I think is related to #1883
when an admin is trying to invite users they will get a message saying they cannot since SMTP is not set and that because emailEnabled is false. SMTP works fine but I use a local server with no username and password.

rocket

That value is set here: https://github.com/RocketChat/Rocket.Chat/blob/develop/packages/rocketchat-ui-admin/admin/users/adminInviteUser.coffee#L4

emailEnabled: ->
        console.log 'emailEnabled', RocketChat.settings.get('MAIL_URL') or (RocketChat.settings.get('SMTP_Host') and RocketChat.settings.get('SMTP_Username') and RocketChat.settings.get('SMTP_Password'))
        return RocketChat.settings.get('MAIL_URL') or (RocketChat.settings.get('SMTP_Host') and RocketChat.settings.get('SMTP_Username') and RocketChat.settings.get('SMTP_Password'))

It still mandates the username and password but I thought the fix in #1904 will at least set the "MAIL_URL" value and that would be enough but it seems both sides return false or nothing!

  • The console returns "emailEnabled" only without false or true

So what do you guys think?
Should we just remove the username & password from this check?
but why MAIL_URL is not seen here? or does this file get executed before that value is set!

@fduraibi
Copy link
Contributor Author

@marceloschmidt can I get your feedback on this?

@marceloschmidt
Copy link
Member

You're right @fduraibi I'll check other places requiring username for smtp server.
Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants