httpslocal / usecases

discussion about use cases
Other
17 stars 11 forks source link

[Requirements] separate and refine requirements #11

Closed tomoyukilabs closed 7 years ago

tomoyukilabs commented 7 years ago

This PR separates Requirements.md from UseCases.md, and refines the requirements in detail. Also, this addresses several comments in https://github.com/httpslocal/usecases/issues/4.

@dajiaji Could you please review this PR? @igarashi50 Any comments and suggestions are welcome.

igarashi50 commented 7 years ago

I think that the "Device Authentication" should be spited in two cases. One is before a certificate grant and issuance is done, the other is after then. The former case depends on the "Certificate Grant and Issuance". I suggest that "Device Authentication" states the following requirements on the latter case.

I suggest that the following requirements about the former case will be be clarified the based on the "Certificate Grant and Issuance". I also think the user grant for "Device Delegate" is optional.

I also suggest including the following requirement in to "Device Discovery" assuming that the "Device Discovery" states the requirements after "Certificate Grand and Issuance" is done.

igarashi50 commented 7 years ago

Alternatively, the following requirement is better for a user on device selection. Any comment?

tomoyukilabs commented 7 years ago

@igarashi50 As you mentioned, this PR only includes "authenticate before grant/issue a certificate" rationale. Although I considered the similar idea at first, I have left this PR as it is now for the following reasons:

I suggest that the following requirements about the former case will be be clarified the based on the "Certificate Grant and Issuance"

Do you mean that typing PIN code is required two times for both authentication and certificate grant/issuance?

I also think the user grant for "Device Delegate" is optional.

I can agree if the device behind the device delegate can request the UA to respond with PIN code via the delegate.

  • UA shall expose a user the information of discovered devices by indicating they have certificates for the endpoint URL obtained by.

This PR intentionally omits this sort of idea since I currently assume that discovery about server-capable devices without a certificate, which could not be connected due to mixed content restriction, would be out of scope.

tomoyukilabs commented 7 years ago

@igarashi50 Excuse me, but I'd like to correct my comment related to device discovery.

  • UA shall expose a user the information of discovered devices by indicating they have certificates for the endpoint URL obtained by.

A device that is not initialized could have no certificate yet. This is the reason why this PR mentions "which has its compliance with certificate grant and issuance and certificate management".

igarashi50 commented 7 years ago

I assume that the "Device Authentication" state the requirements after "Grant and issuance and certification management" is done. The alternative requirement implies that the discovery information contains certification for the endpoint URL. I am OK to leave it out now.

tomoyukilabs commented 7 years ago

Now the draft has been revised overall. @dajiaji @igarashi50 Could you review this again?

tomoyukilabs commented 7 years ago

@dajiaji @igarashi50 Many thanks for discussion and reviewing!