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

[NEW] Add gateway proxy setting. #11124

Closed
wants to merge 2 commits into from
Closed

[NEW] Add gateway proxy setting. #11124

wants to merge 2 commits into from

Conversation

gergoerdosi
Copy link

We use Rocket Chat in a corporate network and would like to send only push notifications through our proxy. This PR adds a new field in push settings that allows to use a proxy.

@Csiffy
Copy link

Csiffy commented Jul 30, 2018

This change would be welcome also for our company, because we use proxy too, and unfortunately currently we do not use notification.

@gergoerdosi
Copy link
Author

Is there anything blocking this from merging?

@@ -1503,6 +1503,18 @@ RocketChat.settings.addGroup('Push', function() {
}
]
});
this.add('Push_gateway_proxy', '', {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if we shouldn't make this more generic. The oembed fetch and a few others also would benefit from a proxy setting

@reetp
Copy link

reetp commented Jun 26, 2019

A bump.

Is there any reason why this hasn't been adopted?

There are lots of queries on using Rocket behind corporate Proxies and precious little help for users.

I understand this is only for push notifications, and some want to use a Corporate Proxy for all traffic so it may not be a cure all.

@geekgonecrazy
Copy link
Contributor

@reetp unfortunately this will not be sufficient. :( The gateway will require a token be passed to it, so several other calls will need to go through the proxy as well. Otherwise the gateway will not be able to accept traffic

@reetp
Copy link

reetp commented Jun 26, 2019

@reetp unfortunately this will not be sufficient. :( The gateway will require a token be passed to it, so several other calls will need to go through the proxy as well. Otherwise the gateway will not be able to accept traffic

Perhaps Rocket ought to make a decision on what to do with Corporate proxies?

For better or worse we see a lot of questions on how to push Rocket through a Corporate proxy/cache. I understand it is a complicated scenario, and may be a security nightmare too boot, and not necessarily best practice.

There is also very little information on the subject one way or another, with vague hints around about using proxies which causes a lot of confusion.

If it is something Rocket doesn't want to officially support it then perhaps some documentation on that fact, along with links to resources where people can read up if they want to support themselves.

If it is something Rocket does want to support then it needs documenting accordingly. Just needs a decision ;-)

I guess this PR can be closed?

@rodrigok
Copy link
Member

We should document the Meteor's setting for proxy https://github.com/meteor/meteor/wiki/Using-Meteor-behind-a-proxy or create internal settings for a generic proxy configuration.

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

Successfully merging this pull request may close these issues.

7 participants