Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update WRP Spec for deviceID #53

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Update WRP Spec for deviceID #53

wants to merge 3 commits into from

Conversation

joe94
Copy link
Member

@joe94 joe94 commented Aug 14, 2020

No description provided.

@joe94 joe94 marked this pull request as draft August 14, 2020 02:20
@joe94 joe94 self-assigned this Aug 14, 2020
@joe94 joe94 linked an issue Aug 14, 2020 that may be closed by this pull request
@joe94 joe94 changed the title Feature/device Update WRP Spec for deviceID Aug 14, 2020
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

source | The device_id name of the device originating the request or response.
dest | The device_id name of the target device of the request or response.
content_type | (optional) The media type of the payload.
source | The device_id name of the device originating the request or response (i.e. dns:tr1d1um.example.net for inbound messages).
Copy link
Member Author

Choose a reason for hiding this comment

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

I wonder if parodus is or plans to use the source field (dns:tr1d1um.example.net) on inbound messages.

Copy link
Contributor

Choose a reason for hiding this comment

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

I thought parodus uses the source at the very least to fill in the destination for the response wrp it sends. I could be wrong.

@joe94 joe94 added the enhancement New feature or request label Aug 14, 2020
content/docs/wrp/simple-messages.md Outdated Show resolved Hide resolved
content/docs/wrp/simple-messages.md Outdated Show resolved Hide resolved
content/docs/wrp/simple-messages.md Outdated Show resolved Hide resolved
source | The device_id name of the device originating the request or response (i.e. dns:tr1d1um.example.net for inbound messages).
dest | The device_id name of the target device of the request or response (i.e. mac:112233445566/config for inbound messages).
device_id | The canonical device ID (i.e. mac:112233445566) involved in this envelope. It defers from source or dest in that no further parsing is required to extract the ID from the value.
content_type | (optional) The media type of the payload. If not specified on inbound messages, `application/octet-stream` is used as default.
Copy link
Contributor

Choose a reason for hiding this comment

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

I thought we were only adding this for events? Is this also being added to request-responses?

Copy link
Contributor

Choose a reason for hiding this comment

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

Nevermind, found the pull request (xmidt-org/webpa-common#508). But isn't it filled in for outbound messages, not inbound?

content/docs/wrp/simple-messages.md Outdated Show resolved Hide resolved
content/docs/wrp/simple-messages.md Outdated Show resolved Hide resolved
content/docs/wrp/simple-messages.md Outdated Show resolved Hide resolved
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update WRP simple messages spec
3 participants