Skip to content

Commit

Permalink
Making Uri proto, microuri support mandatory and longuri support opti…
Browse files Browse the repository at this point in the history
…onal (#110)

* Update uri.proto to support microuri

* Modify UEntity and UResource Ids to be required fields

In order to make microuri support standard and mandatory,
all the "Id" fields need to be required fields and the corresponding
"name" fields have to be optional.

* Incorporating suggestions

Make all the fields optional, to allow long form only and micro form only.
  • Loading branch information
ruchirchauhan authored Apr 9, 2024
1 parent 2ff303c commit 3cefb30
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions up-core-api/uprotocol/uri.proto
Original file line number Diff line number Diff line change
Expand Up @@ -74,8 +74,8 @@ message UAuthority {
// A uE that publishes events is a <b>Service</b> role.<br>
// A uE that consumes events is an <b>Application</b> role.
message UEntity {
string name = 1; // Name of the entity
optional uint32 id = 2; // The numeric ID for the uEntity assigned from the name/number registry
optional string name = 1; // Name of the entity
optional uint32 id = 2; // The numeric ID for the uEntity assigned from the name/number registry
optional uint32 version_major = 3; // optional major version of the uEntity
optional uint32 version_minor = 4; // optional minor version of the uEntity
}
Expand All @@ -91,7 +91,7 @@ message UEntity {
// Resources are unique when prepended with UAuthority that represents the device and
// UEntity that represents the service.
message UResource {
string name = 1; // Name of the resource
optional string name = 1; // Name of the resource
optional string instance = 2; // Instance of the resource
optional string message = 3; // Message type for the resource
optional uint32 id = 4; // Numerical representation of the UResource portion of the UUri
Expand Down

0 comments on commit 3cefb30

Please sign in to comment.