Skip to content

feat: add startup wizard and configurable Call-ID generators#867

Closed
darwvin-dev wants to merge 1 commit into
SIPp:masterfrom
darwvin-dev:master
Closed

feat: add startup wizard and configurable Call-ID generators#867
darwvin-dev wants to merge 1 commit into
SIPp:masterfrom
darwvin-dev:master

Conversation

@darwvin-dev
Copy link
Copy Markdown

Start an interactive startup wizard when SIPp is launched without arguments on an interactive terminal. The wizard helps build common -sn and -sf command lines, only offers options supported by the current build, and preserves the existing help behavior for non-interactive no-argument runs.

Add a new -cid_type option for built-in Call-ID generators while keeping -cid_str as the default behavior. Support UUID, compact UUID, random, and timestamp-based Call-IDs, add unit coverage for each mode, and document the new CLI behavior.

Start an interactive startup wizard when SIPp is launched without arguments on an interactive terminal. The wizard helps build common -sn and -sf command lines, only offers options supported by the current build, and preserves the existing help behavior for non-interactive no-argument runs.

Add a new -cid_type option for built-in Call-ID generators while keeping -cid_str as the default behavior. Support UUID, compact UUID, random, and timestamp-based Call-IDs, add unit coverage for each mode, and document the new CLI behavior.
@wdoekes
Copy link
Copy Markdown
Member

wdoekes commented May 23, 2026

Hi.
Would you consider filing two separate PRs so they can be reviewed on their merits sepeately?
Maybe you can also take the time to document the reasoning for the added features.
Thanks!

@darwvin-dev
Copy link
Copy Markdown
Author

Hi, thanks for the feedback.
Yes, that makes sense. I’m working on splitting the changes into two separate PRs so each feature can be reviewed independently.
I’ll also add more context and document the reasoning behind each change. I’ll open the new PRs soon and link them here.
Thanks!

@darwvin-dev
Copy link
Copy Markdown
Author

Split this combined change into two focused PRs as requested:

Closing this combined PR so each feature can be reviewed on its own merits.

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.

2 participants