search

w3c / automotive

W3C Automotive Working Group Specifications
Other
145 stars 68 forks source link

Proposed texts for initialization, closure of HTTPS and merged signal discovery read with dynamic metadata read on HTTPS parts #485

Closed wonsuk73 closed 11 months ago

wonsuk73 commented 11 months ago

1) This is the draft texts for initialization and closure of session life time management on HTTPS transport protocol 2) I might need to merge signal discovery read with dynamic metadata read on HTTPS parts. Because it need to make the sync with WebSocket part. In addition Signal discovery read can use the both static metadata and dynamic metadata on the request message.

tguild commented 11 months ago

Hi @wonsuk73 ,

@UlfBj and I read through your PR and it makes sense with the exception of the sentence about hostname generally having www prefix as that will not be the case when VISS is on a local, in-vehicle network and unlikely as well if remote vehicle connections are allowed.

wonsuk73 commented 11 months ago

@tguild and @UlfBj Thanks for valuable comment! I revised texts. Could you review on this? In addition if you have any ideas to enhance the description, please let me know!

wonsuk73 commented 11 months ago

@tguild, @UlfBj, could you review on revised one?

UlfBj commented 11 months ago

@wonsuk73, The sentence: A client may send a signal discovery read request to access dynamic metadata. is in conflict with what is said in the static metadata chapter https://raw.githack.com/w3c/automotive/gh-pages/spec/VISSv2_Core.html#static-metadata-request where the term "signal discovery is bound to that type of request. I propose that this sentence is removed from the PR. The initialsization/closure parts look fine to me.

wonsuk73 commented 11 months ago

@UlfBj Thanks for review! I fixed the conflict sentence you mentioned :)