nickva commented on issue #52: URL: https://github.com/apache/couchdb-erlfdb/issues/52#issuecomment-1023442155
Good catch @leonardb. I think we just need to add the option. We even have it as the [dialyzer type](https://github.com/apache/couchdb-erlfdb/blob/main/src/erlfdb_nif.erl#L105-L106). It looks like we validate the network options so it's surprising it didn't blow up at https://github.com/apache/couchdb-erlfdb/blob/main/c_src/main.c#L514 from `return enif_make_badarg(env)` if you try to set that option. https://github.com/apache/couchdb-erlfdb/blob/main/src/erlfdb_nif.erl#L105-L106 @kocolosk is right that we may have to wrangle the correct API versions here if it's not documented or was available only earlier/later we could resort to ifdefs possibly. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
