[kamailio/kamailio] auth: Add option to use SHA-256 instead of MD5 in digest authentication (#628)
This is an alternative solution to pull request: https://github.com/kamailio/kamailio/pull/627 You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/628
-- Commit Summary --
* Merge remote-tracking branch 'refs/remotes/kamailio/master' * Merge remote branch 'upstream/master' * Add new module auth_sha256 based on module auth, but using SHA-256 instead of MD5 for digest authentication. * auth: Add option to use SHA-256 instead of MD5 in digest authentication * auth_sha256: delete module that was integrated in module auth
-- File Changes --
M modules/auth/Makefile (2) M modules/auth/api.c (9) M modules/auth/auth_mod.c (29) M modules/auth/auth_mod.h (6) M modules/auth/doc/auth_params.xml (26) M modules/auth/rfc2617.c (4) M modules/auth/rfc2617.h (4) A modules/auth/rfc2617_sha256.c (148) A modules/auth/rfc2617_sha256.h (71)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/628.patch https://github.com/kamailio/kamailio/pull/628.diff
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/628
Thanks for the contribution!
Looking at the sequence of operations, only the patch 4a076a0 needs to be reviewed and merged, right?
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/628#issuecomment-219927265
Hi. Yes, only patch 4a076a0 is relevant.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/628#issuecomment-219939844
Hi. Did anyone look at the proposed feature ? And do you have an idea if and when you might accept this pull request ?
Regards, Andreas
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/628#issuecomment-220553679
These days many of us are at Kamailio World Conference in Berlin, therefore some low activity around the online resources of the project.
This looks like something I will have to review, but being one of the organizers these days I don't have much time to even breed. However, I will try to do it soon.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/628#issuecomment-220586007
Ok, no problem.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/628#issuecomment-220586874
Thanks, the patch was pushed manually on your behalf with commit e7faf1e .
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/628#issuecomment-220933425
Closed #628.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/628#event-668708693
-
AndreasHuber-CH -
Daniel-Constantin Mierla