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

Log old python #54

Merged
merged 6 commits into from Oct 14, 2013
Merged

Log old python #54

merged 6 commits into from Oct 14, 2013

Conversation

@MichaelBlume
Copy link
Contributor

@MichaelBlume MichaelBlume commented Oct 10, 2013

Make sure script parses under python 2.5 and prints an error message and logs something to loggly.

if json is None:
version = get_python_version_string()
log_msg('''{"python_version": "%s", "subdomain": "%s"}''' %
(version, options.subdomain))

This comment has been minimized.

@MichaelBlume

MichaelBlume Oct 10, 2013
Author Contributor

have to generate json manually here because json is not in Python 2.5 stdlib

philip-loggly added a commit that referenced this pull request Oct 14, 2013
@philip-loggly philip-loggly merged commit 9bcfbf2 into loggly:master Oct 14, 2013
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

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