Skip to content

Add parseJson function #64

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Feb 27, 2015
Merged

Add parseJson function #64

merged 5 commits into from
Feb 27, 2015

Conversation

md5
Copy link
Contributor

@md5 md5 commented Feb 26, 2015

Adds the parseJsonArray and parseJsonObject functions. Useful if the you have an environment variable like MYCONFIG='{"enabled":true,"features":["foo","bar"]}'. I was inspired by INTERLOCK_CONFIG in https://github.com/ehazlett/interlock

This includes the changes from #63.

md5 and others added 3 commits February 25, 2015 18:17
@md5
Add newTemplate function
ef19866 
Creates a new text.Template with the nginx-proxy FuncMap configured
@md5
Remove reflection from TestKeys
ad7451f 
Now using an actual template. Most tests should probably do this.
@md5
Add parseJsonArray and parseJsonObject
2fb6a6a
@md5
Copy link
Contributor Author

md5 commented Feb 27, 2015

I think this can actually be a single parseJson function. Closing for the moment.

@md5 md5 closed this Feb 27, 2015
@md5 md5 changed the title Add functions to parse JSON Add parseJson function Feb 27, 2015
@md5 md5 reopened this Feb 27, 2015
jwilder added a commit that referenced this pull request Feb 27, 2015
@jwilder
Merge pull request #64 from md5/add-function-parse-json
9978084 
Add parseJson function
@jwilder jwilder merged commit 9978084 into nginx-proxy:master Feb 27, 2015
@jwilder
Copy link
Collaborator

jwilder commented Feb 27, 2015

Thanks!

@md5 md5 deleted the add-function-parse-json branch February 28, 2015 00:18
@md5
Copy link
Contributor Author

md5 commented Feb 28, 2015

🍻

@md5 md5 mentioned this pull request Dec 4, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@md5 @jwilder