Skip to content

Allow command flags to be used when constructing new message#23

Merged
lwlee2608 merged 3 commits intoMATRIXXSoftware:mainfrom
kwokfu:feat-command-flags
Apr 10, 2025
Merged

Allow command flags to be used when constructing new message#23
lwlee2608 merged 3 commits intoMATRIXXSoftware:mainfrom
kwokfu:feat-command-flags

Conversation

@kwokfu
Copy link
Contributor

@kwokfu kwokfu commented Apr 10, 2025

NewMessage() now accept command flags.

e.g. to create a proxiable request

import { cmd, cmdFlag, app } from './diam/const.js';

diam.newMessage(cmd.CreditControl, app.ChargingControl, cmdFlag.Request | cmdFlag.Proxiable);

Introduce NewRequest() for message to have the R-bit pre-configured.

e.g. to create a request

import { cmd, app } from './diam/const.js';

diam.newRequest(cmd.CreditControl, app.ChargingControl);

Fixes #22.

Copy link
Contributor

@lwlee2608 lwlee2608 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lwlee2608 lwlee2608 merged commit 0aef490 into MATRIXXSoftware:main Apr 10, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support for adding flags inside header while making diameter request

2 participants