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

Allow logging to be delegated to an arbitrary Logger #223

Merged
merged 3 commits into from
Jun 30, 2015

Conversation

ghost
Copy link

@ghost ghost commented Apr 24, 2015

No description provided.

@ghost
Copy link
Author

ghost commented May 2, 2015

@pitbulk - any objection to this change? It seems harmless, and allows for smoother integration with the host app's logging.

@Lordnibbler
Copy link
Contributor

@tony-spataro-rs thanks for this contribution -- I'm ok with it if you could just add some information to the readme on how to use this new logging feature.

Thanks

@ghost
Copy link
Author

ghost commented May 14, 2015

@Lordnibbler done; :shipit: !

@ghost ghost force-pushed the configurable_logging branch from 1cf4ef8 to 393c0e7 Compare May 19, 2015 00:21
@ghost
Copy link
Author

ghost commented Jun 9, 2015

@Lordnibbler or @pitbulk - any chance of getting this merged before the next release?

@Lordnibbler
Copy link
Contributor

thanks, unfortunately in order to merge this branch we will need to rebase it off of master. can you do this @tony-spataro-rs? then we can release it as part of v1.0

@stevecrozz
Copy link
Contributor

@Lordnibbler I rebased these changes onto master

pitbulk added a commit that referenced this pull request Jun 30, 2015
Allow logging to be delegated to an arbitrary Logger
@pitbulk pitbulk merged commit 1737ebb into SAML-Toolkits:master Jun 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants