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

Authentication and Authorization chapter #314

Merged
merged 1 commit into from Nov 19, 2019
Merged

Conversation

danielwilms
Copy link
Contributor

@danielwilms danielwilms commented Nov 6, 2019

CORE: created with input from VIWI.
TRANSPORT: updated to match current VSS modelling.

Readable CORE document
Readable TRANSPORT document

Fixes #308

…t from VIWI. TRANSPORT doc updated to match current VSS modelling.
@UlfBj UlfBj added the VISS v2 Generation Two of the spec label Nov 6, 2019
Copy link
Contributor

@UlfBj UlfBj left a comment

Choose a reason for hiding this comment

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

+1

@PatrickCQ
Copy link
Contributor

Usage of Transport Document unclear to me.
See issue #313

@UlfBj
Copy link
Contributor

UlfBj commented Nov 7, 2019

Usage of Transport Document unclear to me.

In #313 I think you clarify the Transport document usage correctly yourself, so what is unclear?
Gen2 specifies two transport protocols, HTTP and WebSocket, and the specifics of applying the Gen2 messaging to each of these protocols needs to be described somewhere. @PatrickCQ Would you rather see it all in one document?

@PatrickCQ
Copy link
Contributor

In #313 I think you clarify the Transport document usage correctly yourself, so what is unclear?
Gen2 specifies two transport protocols, HTTP and WebSocket, and the specifics of applying the Gen2 messaging to each of these protocols needs to be described somewhere. @PatrickCQ Would you rather see it all in one document?

I would like to see it in 3 separate documents as summarized in #313.

@UlfBj
Copy link
Contributor

UlfBj commented Nov 7, 2019

I would like to see it in 3 separate documents as summarized in #313.

That is an opinion that is suitable for discussion in the WG, it has not much to do with this PR.

Copy link
Contributor

@PatrickCQ PatrickCQ left a comment

Choose a reason for hiding this comment

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

Please add changes to the corresponding documents as discussed https://lists.w3.org/Archives/Member/member-automotive/2018Nov/0006.html

@UlfBj
Copy link
Contributor

UlfBj commented Nov 7, 2019

Please add changes to the corresponding documents as discussed https://lists.w3.org/Archives/Member/member-automotive/2018Nov/0006.html

That is a long mail thread, I think you need to be more specific about what changes mentioned there you think should be added.

@PatrickCQ
Copy link
Contributor

Please add changes to the corresponding documents as discussed https://lists.w3.org/Archives/Member/member-automotive/2018Nov/0006.html

That is a long mail thread, I think you need to be more specific about what changes mentioned there you think should be added.

Please use three documents:

  • Core: as is
  • Transport: with general requirements to a transport protocol
  • Transport_HTTPS.WS (new): to describe the specific handling of the requirements from Transport for the HTTP and WS combination

@danielwilms
Copy link
Contributor Author

Hi guys,

I try to jump in as moderator, if you allow :) @PatrickCQ, please mark in the document, which part you want to see in which document. If it's clear to you then please add something like

start <document where it should go to>
...
end  <document where it should go to>

Then it's much more efficient to discuss it. If @UlfBj agrees, he can move it. If not we bring it up in the call. We have to remember, that the others agreed already to the change, with the issues, we raised in the #309.

@UlfBj
Copy link
Contributor

UlfBj commented Nov 7, 2019

Please use three documents:
Core: as is
Transport: with general requirements to a transport protocol
Transport_HTTPS.WS (new): to describe the specific handling of the requirements from Transport for the HTTP and WS combination

I think this should be discussed in the WG first. If I am not mistaken the WG recently decided to go with the existing Gen2 documents, so if that decision should be changed it needs to be discussed by the WG first.
I think you are bringing this up in the wrong context, it is a WG issue and is not related to this PR.

@PatrickCQ
Copy link
Contributor

I think you are bringing this up in the wrong context, it is a WG issue and is not related to this PR.

I agree. #313 can be closed with the conclusion from the the discussion in the working group.

@UlfBj
Copy link
Contributor

UlfBj commented Nov 7, 2019

I think you are bringing this up in the wrong context, it is a WG issue and is not related to this PR.
I agree.

With that statement I assume you will remove your change request and approve this PR.

@MagnusGun
Copy link

@PatrickCQ, I don't understand the logic of separating the spec into multiple documents can you please elaborate

@PatrickCQ
Copy link
Contributor

@PatrickCQ, I don't understand the logic of separating the spec into multiple documents can you please elaborate

please see #313

@MagnusGun
Copy link

@PatrickCQ, I don't understand the logic of separating the spec into multiple documents can you please elaborate

please see #313

I have read #313 and I don't have any problems with the separation of topics but I dont get why we need to have them as separate documents instead of chapters in the same document...

@tguild
Copy link
Member

tguild commented Nov 12, 2019

@MagnusGun @PatrickCQ whether to split out into transport protocol specific documents should depend on how much clarification is needed for each. I would mildly prefer moderate sized chapters to additional documents. Large chapters can indeed become unwieldy and warrant spinning off separate documents. It is possible to defer and separate later unless we have an early indication the HTTP/WS specific aspects will be sizable

@tguild tguild merged commit 07329af into w3c:gh-pages Nov 19, 2019
@tguild
Copy link
Member

tguild commented Nov 19, 2019

https://www.w3.org/2019/11/19-auto-minutes

issue whether to have chapter or separate protocol documents in issue 313

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
VISS v2 Generation Two of the spec
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Gen2 feature: Authentication and authorization
5 participants