CONTRIBUTING.md

Contributing

• Contributing with Issues
• Contributing with Code
  • C/C++
    • Code Style
    • Linting
  • Typescript / Javascript
    • Code Style
    • Linting
  • Setup
  • Adding New libcurl Options
  • Changing libcurl Version Used on Prebuilt Binaries for Windows
  • Debugging with lldb
  • Publishing New Releases
    • Build Matrix

Contributing with Issues

Before opening an issue try to search the existing ones for the same problem.

Make sure to include on your issue the following information:

• Node.js Version
• npm version
• Operational System (name and version)
• Package version
• Logs of the installation

Contributing with Code

The addon lib code is written in Typescript, while the addon itself is written in C++.

Folders ./scripts and ./tools contain scripts in Javascript, those are used mostly during installation and on CI.

C/C++

Code Style

C/C++ code is written following Google style guide (with some minor changes), and clang-format can/should be used to automatically format the code. There is already a .clang-format on the repository.

Linting

cpplint is used to lint C/C++ code

Typescript / Javascript

Code Style

TS/JS code should be formatted using prettier

Linting

ts-lint is used to lint TS code, while JS code is using eslint.

Setup

If on Windows, first you will need to grab the deps:

node scripts/update-deps.js

Install the dependencies, this will also build the addon:

npm install

If you made some change to the C++ code, you can just build the addon again:

npm run pregyp build

In case you need to rebuild:

npm run pregyp rebuild

If you have any issues with the build process, please refer to a readme build troubleshooting section.

Adding New libcurl Options

If you want to include a new libcurl option on the addon, those are the basic steps:

1. Add the option to their correct category on Curl.cc There are different vectors there for the expected type of the option's value, like curlOptionInteger, curlOptionString, etc. Make sure to use a #if directive to include this option only if building against a libcurl version that supports it, otherwise there will be an compilation error on older versions.
2. In case the option expects a value of type other than number | string | boolean, you must also add it to their respective key on the object optionKindMap inside ./scripts/data/options.js. In case you add it to the other key, which means this option has a specific value, you must also add the option expected value type to the object optionKindValueMap right below, on that same file.
3. If the option may use an enum, you should also create or update the existing enum. For example, libcurl 7.75.0 added AWS Sig v4 authentication method, for that a new option and constant were added in libcurl, CURLOPT_AWS_SIGV4 and CURLAUTH_AWS_SIGV4. To add support for that option we:
   • Added the CURLOPT_AWS_SIGV4 constant as the AwsSigV4 member in the CurlAuth enum. To get the value we looked at the libcurl source code.
   • Added the CURLOPT_AWS_SIGV4 as a string option. Full commit with the above changes is available here.
4. Run node ./scripts/build-constants.js, this will generate an updated list of options on ./lib/generated/, and also update the files [./lib/Curl.ts] and [./lib/EasyNativeBinding.ts] with overloads for the setOpt method. Make sure the options added are correct.
5. If running the above adds extra options that you do not want to add / are not related to the options you are adding, please feel free to remove them manually from the generated output. We will try to improve this experience later, but for now you have to manually remove them.

Changing libcurl Version Used on Prebuilt Binaries for Windows

You will need to open a PR against the repository JCMais/curl-for-windows upgrading libcurl there.

After that a new tag will be created on this repo, which we can them use on the file LIBCURL_VERSION_WIN_DEPS.

Debugging with lldb

1. Install lldb On Debian based linux:

sudo apt-get install lldb

2. Install Node.js lldb plugin:

npm i -g llnode

3. Run script that causes core dump

llnode -- /path/to/bin/node --abort_on_uncaught_exception script.js

4. Profit

More information go to https://github.com/nodejs/llnode

Publishing New Releases

1. Make PR and merge changes into develop
2. Manually run github action "Release new version of node-libcurl"
3. Set version to manually and run
4. a Release should appear and if the build-and-release job is still running the assets will appear when ready

Build Matrix

GitHub Actions:

• Node.js (Linux, macOS)
• Electron (macOS)

GitHub Actions are also used to lint PRs, for that a build runs on Linux.