Chris Barth
Chris Barth
@shunkica , I was just looking back at some PRs to see if we could get them landed. Let me know what you think about my last comment.
Thank you for putting this up. Please include some tests so that this doesn't break in the future. Also, please make sure that you run `npm test` and `npm run...
Can you please link to the spec?
@srd90 , you're always on top of this stuff :) Thanks! @imedacf , please feel free to put up a PR that aligns with this spec and we'll review it.
Can you site where in the documentation it mentions that default value, or, even better, put up a PR with a test that makes sure the defaults correspond to the...
@LoneRifle , do you have any thoughts on this? In all the refactoring, I don't think we touched anything related to any of these defaults in the code, but I'd...
@CarlaMck77 , we're more than happy to help anyone through the process of submitting PRs. Having said that, in harmony with the approach taken with `node-saml` and `passport-saml`, we are...
@CarlaMck77 Again, thanks for pointing this out and @srd90 , thank you for providing specific details. I believe with the latest PRs that have landed there are no longer any...
This seems reasonable. If possible, please reference the spec on this and feel free to put up a PR that will do this. It seems that if the `doc` is...
@cyberrspiritt , if that solution worked for you, would you mind creating a PR with a test suite and making a PR so that the community can benefit and so...