telefonicaid / telefonicaid/iotagent-node-lib

Remove dependency on deprecated domain module

Open
#1,585 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

help wanted Techdeb
Dominant language
JavaScript
Stars
62
Forks
90
Avg merge
2h 35m
Merged PRs (30d)
1

Description

Component

Techdeb

Is your feature request related to a problem? Please describe

https://nodejs.org/docs/latest-v18.x/api/domain.html#domain

Stability: 0 - Deprecated

Source Code: lib/domain.js

This module is pending deprecation. Once a replacement API has been finalized, this module will be fully deprecated. Most developers should not have cause to use this module. Users who absolutely must have the functionality that domains provide may rely on it for the time being but should expect to have to migrate to a different solution in the future.

Domains provide a way to handle multiple different IO operations as a single group. If any of the event emitters or callbacks registered to a domain emit an 'error' event, or throw an error, then the domain object will be notified, rather than losing the context of the error in the process.on('uncaughtException') handler, or causing the program to

Describe the solution you'd like

There are some approaches to use, but basically is include logger into current request.

Describe alternatives you've considered

https://github.com/telefonicaid/fiware-pep-steelskin/pull/528
https://github.com/telefonicaid/iotagent-manager/pull/327

The systemd version you checked that didn't have the feature you are asking for

No response

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the library's current use of Node's deprecated domain module and reviewing the linked alternative pull requests. Compare the proposed approach of including a logger in the current request, then confirm the replacement preserves the needed request and error context without the domain dependency.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, node.js
Domain
backend
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.