elerch / SAML2

Fork of SAML2 library on codeplex. This library removes dependencies on Asp.net
Mozilla Public License 2.0
117 stars 62 forks source link

Append Url with & and query string instead of ? if QS already present #8

Closed jbparker closed 8 years ago

jbparker commented 8 years ago

Hi,

I went the dummy route here (didn't add to instance functions / properties / utils) and simply replaced the problem areas with conditional evaluations to determine whether to append query string with ? or &.

If there is a better place to put some abstraction of this behavior in the library, let me know and I will fix this.

Resolves Issue #7.

elerch commented 8 years ago

Thanks for the contribution!

jbparker commented 8 years ago

My pleasure - thanks for accepting / reviewing it!