swift-server / swift-kafka-client

Apache License 2.0
81 stars 20 forks source link

Error CNIOBoringSSL_HMAC with swift 6 on ubuntu #173

Open ZirgVoice opened 1 day ago

ZirgVoice commented 1 day ago

When trying to build a project in docker on ubuntu with swift 6 we get the following error:

37.89 /build/.build/checkouts/swift-kafka-client/Sources/Crdkafka/librdkafka/src/rdkafka_ssl.c:1873:14: error: call to undeclared function 'CNIOBoringSSL_HMAC'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
37.89  1873 |         if (!HMAC(evp, (const unsigned char *)in->ptr, (int)in->size, saltplus,
37.89       |              ^
37.89 /build/.build/checkouts/swift-nio-ssl/Sources/CNIOBoringSSL/include/CNIOBoringSSL_boringssl_prefix_symbols.h:1311:14: note: expanded from macro 'HMAC'
37.89  1311 | #define HMAC BORINGSSL_ADD_PREFIX(BORINGSSL_PREFIX, HMAC)
37.89       |              ^
37.89 /build/.build/checkouts/swift-nio-ssl/Sources/CNIOBoringSSL/include/CNIOBoringSSL_boringssl_prefix_symbols.h:17:36: note: expanded from macro 'BORINGSSL_ADD_PREFIX'
37.89    17 | #define BORINGSSL_ADD_PREFIX(a, b) BORINGSSL_ADD_PREFIX_INNER(a, b)
37.89       |                                    ^
37.89 /build/.build/checkouts/swift-nio-ssl/Sources/CNIOBoringSSL/include/CNIOBoringSSL_boringssl_prefix_symbols.h:18:42: note: expanded from macro 'BORINGSSL_ADD_PREFIX_INNER'
37.89    18 | #define BORINGSSL_ADD_PREFIX_INNER(a, b) a ## _ ## b
37.89       |                                          ^
37.89 <scratch space>:76:1: note: expanded from here
37.89    76 | CNIOBoringSSL_HMAC
37.89       | ^
37.89 /build/.build/checkouts/swift-kafka-client/Sources/Crdkafka/librdkafka/src/rdkafka_ssl.c:1873:14: note: did you mean 'CNIOBoringSSL_SHA1'?
37.89 /build/.build/checkouts/swift-nio-ssl/Sources/CNIOBoringSSL/include/CNIOBoringSSL_boringssl_prefix_symbols.h:1311:14: note: expanded from macro 'HMAC'
37.89  1311 | #define HMAC BORINGSSL_ADD_PREFIX(BORINGSSL_PREFIX, HMAC)
37.89       |              ^
37.89 /build/.build/checkouts/swift-nio-ssl/Sources/CNIOBoringSSL/include/CNIOBoringSSL_boringssl_prefix_symbols.h:17:36: note: expanded from macro 'BORINGSSL_ADD_PREFIX'
37.89    17 | #define BORINGSSL_ADD_PREFIX(a, b) BORINGSSL_ADD_PREFIX_INNER(a, b)
37.89       |                                    ^
37.89 /build/.build/checkouts/swift-nio-ssl/Sources/CNIOBoringSSL/include/CNIOBoringSSL_boringssl_prefix_symbols.h:18:42: note: expanded from macro 'BORINGSSL_ADD_PREFIX_INNER'
37.89    18 | #define BORINGSSL_ADD_PREFIX_INNER(a, b) a ## _ ## b
37.89       |                                          ^
37.89 <scratch space>:76:1: note: expanded from here
37.89    76 | CNIOBoringSSL_HMAC
37.89       | ^
37.89 /build/.build/checkouts/swift-nio-ssl/Sources/CNIOBoringSSL/include/CNIOBoringSSL_sha.h:90:25: note: 'CNIOBoringSSL_SHA1' declared here
37.89    90 | OPENSSL_EXPORT uint8_t *SHA1(const uint8_t *data, size_t len,
37.89       |                         ^
37.89 /build/.build/checkouts/swift-nio-ssl/Sources/CNIOBoringSSL/include/CNIOBoringSSL_boringssl_prefix_symbols.h:1727:14: note: expanded from macro 'SHA1'
37.89  1727 | #define SHA1 BORINGSSL_ADD_PREFIX(BORINGSSL_PREFIX, SHA1)
37.89       |              ^
37.89 /build/.build/checkouts/swift-nio-ssl/Sources/CNIOBoringSSL/include/CNIOBoringSSL_boringssl_prefix_symbols.h:17:36: note: expanded from macro 'BORINGSSL_ADD_PREFIX'
37.89    17 | #define BORINGSSL_ADD_PREFIX(a, b) BORINGSSL_ADD_PREFIX_INNER(a, b)
37.89       |                                    ^
37.89 /build/.build/checkouts/swift-nio-ssl/Sources/CNIOBoringSSL/include/CNIOBoringSSL_boringssl_prefix_symbols.h:18:42: note: expanded from macro 'BORINGSSL_ADD_PREFIX_INNER'
37.89    18 | #define BORINGSSL_ADD_PREFIX_INNER(a, b) a ## _ ## b
37.89       |                                          ^
37.89 <scratch space>:163:1: note: expanded from here
37.89   163 | CNIOBoringSSL_SHA1
37.89       | ^
37.89 1 error generated.
blindspotbounty commented 1 day ago

I've opened a PR for this issue: https://github.com/swift-server/swift-kafka-client/pull/174 Hope to merge it soon