Can sha function changed to optional #9

Closed
runner-mei opened this Issue Apr 5, 2012 · 6 comments

Comments

Projects
None yet
2 participants

在 src/ngx_http_set_hash.c 中引用了 ngx_sha1.h 头文件,它依赖于 openssl, 在 nginx 中它是可选依赖, 有一个宏(NGX_HAVE_SHA1)来判断是否有 SHA,

你能不能将你的代码改一下,我编译时没有加 openssl, 因此报错了。
非常感谢你的工作。
不好意思,我又给您添麻烦了。

Owner

agentzh commented Apr 7, 2012

多谢报告 :) 您能提供一个小补丁么?我直接应用一下,呵呵。

agentzh added a commit that referenced this issue Apr 7, 2012

bugfix: we should omit the set_sha1 directive when we do not have any…
… SHA1 libraries (including OpenSSL) installed. thanks runner-mei for reporting this in github issue #9.
Owner

agentzh commented Apr 7, 2012

补丁已经提交到 master 分支。请在你那儿再尝试一下看看是否可以了?再次感谢!

有点小问题, 编译时src/ngx_http_set_hash.c 第25 行出错, 报告找不到 SHA_DIGEST_LENGTH 的定义,

Owner

agentzh commented Apr 11, 2012

agentzh added a commit that referenced this issue Apr 17, 2012

bugfix: compilation error on SHA_DIGEST_LENGTH when sha1 library is m…
…issing. thanks runner-mei for reporting in github issue #9.
Owner

agentzh commented Apr 17, 2012

我刚刚提交了修复。能否再尝试一下?非常感谢!

好了,谢谢。

@runner-mei runner-mei closed this Apr 18, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment