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

Proxy domain support #27

Merged
merged 3 commits into from Aug 3, 2015
Merged

Proxy domain support #27

merged 3 commits into from Aug 3, 2015

Conversation

@varshneyjayant
Copy link
Contributor

@varshneyjayant varshneyjayant commented Jul 31, 2015

No description provided.

'logglyKey': '8c518f97-e3e0-4bfb-a8ed-582d084a5289',
'sendConsoleErrors' : true,
'tag' : 'javascript-logs',
'useDomainProxy' : true

This comment has been minimized.

@mostlyjason

mostlyjason Jul 31, 2015
Contributor

Why not just use the domain proxy if it's not null? Wouldn't it be simpler to have one variable instead of two?

+_LTracker.push({

  • 'logglyKey': '8c518f97-e3e0-4bfb-a8ed-582d084a5289',
  • 'sendConsoleErrors' : true,
  • 'tag' : 'javascript-logs',
  • 'domainProxy' : 'loggly.mydomain.com'
    +});

This comment has been minimized.

@varshneyjayant

varshneyjayant Aug 3, 2015
Author Contributor

Are we supporting them to use any domain? We are currently appending /loggly to their current domain. Like

inputUrl = window.location.host + '/loggly' + '/input/token...;

Also, we have give the example for the same in server configuration.

This comment has been minimized.

@mostlyjason

mostlyjason Aug 3, 2015
Contributor

Ahh thanks for explaining! That seems fine too. If someone wants to add other domains they can do it in another PR.

vhalbwachs added a commit that referenced this pull request Aug 3, 2015
@vhalbwachs vhalbwachs merged commit 5115e4d into loggly:master Aug 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.