search

knastase / simplesamlphp-duosecurity

Two factor authentication module using Duo Security for SimpleSAMLphp
MIT License
14 stars 7 forks source link

updating duo_web.php after format injection fix #1

Closed erikbarbara closed 9 years ago

erikbarbara commented 9 years ago

Updating duo_web.php with the most recent version at https://github.com/duosecurity/duo_php. There was a format injection vulnerability patched a month ago.

knastase commented 9 years ago

Thanks Erik!

erikbarbara commented 9 years ago

Hi Kevin,

I noticed that there's no license on this repo. Would you consider adding one so it's clearer how end-users can use it? Maybe an MIT license?

Thanks!

Erik

On Tue, Mar 17, 2015 at 4:43 PM, Kevin Nastase notifications@github.com wrote:

Thanks Erik!

— Reply to this email directly or view it on GitHub https://github.com/knasty51/simplesamlphp-duosecurity/pull/1#issuecomment-82590307 .

knastase commented 9 years ago

Sure thing! MIT license added.

On Tue, Mar 17, 2015 at 4:53 PM, Erik Barbara notifications@github.com wrote:

Hi Kevin,

I noticed that there's no license on this repo. Would you consider adding one so it's clearer how end-users can use it? Maybe an MIT license?

Thanks!

Erik

On Tue, Mar 17, 2015 at 4:43 PM, Kevin Nastase notifications@github.com wrote:

Thanks Erik!

— Reply to this email directly or view it on GitHub < https://github.com/knasty51/simplesamlphp-duosecurity/pull/1#issuecomment-82590307

.

— Reply to this email directly or view it on GitHub https://github.com/knasty51/simplesamlphp-duosecurity/pull/1#issuecomment-82596223 .

erikbarbara commented 9 years ago

Great. I folded your license in to my fork. :)

On Wed, Mar 18, 2015 at 9:10 AM, Kevin Nastase notifications@github.com wrote:

Sure thing! MIT license added.

On Tue, Mar 17, 2015 at 4:53 PM, Erik Barbara notifications@github.com wrote:

Hi Kevin,

I noticed that there's no license on this repo. Would you consider adding one so it's clearer how end-users can use it? Maybe an MIT license?

Thanks!

Erik

On Tue, Mar 17, 2015 at 4:43 PM, Kevin Nastase <notifications@github.com

wrote:

Thanks Erik!

— Reply to this email directly or view it on GitHub <

https://github.com/knasty51/simplesamlphp-duosecurity/pull/1#issuecomment-82590307

.

— Reply to this email directly or view it on GitHub < https://github.com/knasty51/simplesamlphp-duosecurity/pull/1#issuecomment-82596223

.

— Reply to this email directly or view it on GitHub https://github.com/knasty51/simplesamlphp-duosecurity/pull/1#issuecomment-82963510 .