Skip to content

Update api.md#1715

Merged
fgalan merged 2 commits into
masterfrom
task/allow_headers_in_command
May 20, 2025
Merged

Update api.md#1715
fgalan merged 2 commits into
masterfrom
task/allow_headers_in_command

Conversation

@AlvaroVega

@AlvaroVega AlvaroVega commented May 20, 2025

Copy link
Copy Markdown
Member

Continuation #1714

@AlvaroVega AlvaroVega requested a review from fgalan May 20, 2025 06:46
Comment thread doc/api.md
particular IOTAs documentation for allowed values of this field in each case.
- **contentType**: `content-type` header used when send command by HTTP transport (ignored in other kinds of
transports)
- **headers**: extra customer headers used when send command by HTTP transport (ignored in other kinds of

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe we should explain that the value of this field is a map which keys are header names and values are jexl expressions? In addition, maybe an example could help.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Done in 14dc6b0

@fgalan fgalan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@fgalan fgalan merged commit a3023c8 into master May 20, 2025
7 checks passed
@fgalan fgalan deleted the task/allow_headers_in_command branch May 20, 2025 08:09
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