Skip to content

Commit

Permalink
Add/improve javadoc on required builder methods
Browse files Browse the repository at this point in the history
  • Loading branch information
emlun committed Sep 28, 2021
1 parent 0763da9 commit d8e2d60
Show file tree
Hide file tree
Showing 16 changed files with 162 additions and 4 deletions.
1 change: 1 addition & 0 deletions NEWS
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@ Fixes:

* Added missing JavaDoc for `id` and `name` methods of initial
`RelyingPartyIdentityBuilder` stages.
* Added and improved JavaDoc for required builder methods.


== Version 1.10.1 ==
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -148,6 +148,9 @@ public static class MandatoryStages {
* {@link
* AssertionRequestBuilder#publicKeyCredentialRequestOptions(PublicKeyCredentialRequestOptions)
* publicKeyCredentialRequestOptions} is a required parameter.
*
* @see
* AssertionRequestBuilder#publicKeyCredentialRequestOptions(PublicKeyCredentialRequestOptions)
*/
public AssertionRequestBuilder publicKeyCredentialRequestOptions(
PublicKeyCredentialRequestOptions publicKeyCredentialRequestOptions) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -79,12 +79,24 @@ public static class FinishAssertionOptionsBuilder {
public static class MandatoryStages {
private final FinishAssertionOptionsBuilder builder = new FinishAssertionOptionsBuilder();

/**
* {@link FinishAssertionOptionsBuilder#request(AssertionRequest) request} is a required
* parameter.
*
* @see FinishAssertionOptionsBuilder#request(AssertionRequest)
*/
public Step2 request(AssertionRequest request) {
builder.request(request);
return new Step2();
}

public class Step2 {
/**
* {@link FinishAssertionOptionsBuilder#response(PublicKeyCredential) response} is a
* required parameter.
*
* @see FinishAssertionOptionsBuilder#response(PublicKeyCredential)
*/
public FinishAssertionOptionsBuilder response(
PublicKeyCredential<AuthenticatorAssertionResponse, ClientAssertionExtensionOutputs>
response) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -82,12 +82,24 @@ public static class MandatoryStages {
private final FinishRegistrationOptionsBuilder builder =
new FinishRegistrationOptionsBuilder();

/**
* {@link FinishRegistrationOptionsBuilder#request(PublicKeyCredentialCreationOptions)
* request} is a required parameter.
*
* @see FinishRegistrationOptionsBuilder#request(PublicKeyCredentialCreationOptions)
*/
public Step2 request(PublicKeyCredentialCreationOptions request) {
builder.request(request);
return new Step2();
}

public class Step2 {
/**
* {@link FinishRegistrationOptionsBuilder#response(PublicKeyCredential) response} is a
* required parameter.
*
* @see FinishRegistrationOptionsBuilder#response(PublicKeyCredential)
*/
public FinishRegistrationOptionsBuilder response(
PublicKeyCredential<
AuthenticatorAttestationResponse, ClientRegistrationExtensionOutputs>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -114,19 +114,37 @@ public static class RegisteredCredentialBuilder {
public static class MandatoryStages {
private RegisteredCredentialBuilder builder = new RegisteredCredentialBuilder();

/**
* {@link RegisteredCredentialBuilder#credentialId(ByteArray) credentialId} is a required
* parameter.
*
* @see RegisteredCredentialBuilder#credentialId(ByteArray)
*/
public Step2 credentialId(ByteArray credentialId) {
builder.credentialId(credentialId);
return new Step2();
}

public class Step2 {
/**
* {@link RegisteredCredentialBuilder#userHandle(ByteArray) userHandle} is a required
* parameter.
*
* @see RegisteredCredentialBuilder#userHandle(ByteArray)
*/
public Step3 userHandle(ByteArray userHandle) {
builder.userHandle(userHandle);
return new Step3();
}
}

public class Step3 {
/**
* {@link RegisteredCredentialBuilder#publicKeyCose(ByteArray) publicKeyCose} is a required
* parameter.
*
* @see RegisteredCredentialBuilder#publicKeyCose(ByteArray)
*/
public RegisteredCredentialBuilder publicKeyCose(ByteArray publicKeyCose) {
return builder.publicKeyCose(publicKeyCose);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,11 @@ public static class StartRegistrationOptionsBuilder {
public static class MandatoryStages {
private final StartRegistrationOptionsBuilder builder = new StartRegistrationOptionsBuilder();

/**
* {@link StartRegistrationOptionsBuilder#user(UserIdentity) user} is a required parameter.
*
* @see StartRegistrationOptionsBuilder#user(UserIdentity)
*/
public StartRegistrationOptionsBuilder user(UserIdentity user) {
return builder.user(user);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,11 @@ public static class AttestationBuilder {
public static class MandatoryStages {
private final AttestationBuilder builder = new AttestationBuilder();

/**
* {@link AttestationBuilder#trusted(boolean) trusted} is a required parameter.
*
* @see AttestationBuilder#trusted(boolean)
*/
public AttestationBuilder trusted(boolean trusted) {
return builder.trusted(trusted);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -112,19 +112,37 @@ public static class MandatoryStages {
private final AuthenticatorAssertionResponseBuilder builder =
new AuthenticatorAssertionResponseBuilder();

/**
* {@link AuthenticatorAssertionResponseBuilder#authenticatorData(ByteArray)
* authenticatorData} is a required parameter.
*
* @see AuthenticatorAssertionResponseBuilder#authenticatorData(ByteArray)
*/
public Step2 authenticatorData(ByteArray authenticatorData) {
builder.authenticatorData(authenticatorData);
return new Step2();
}

public class Step2 {
/**
* {@link AuthenticatorAssertionResponseBuilder#clientDataJSON(ByteArray) clientDataJSON} is
* a required parameter.
*
* @see AuthenticatorAssertionResponseBuilder#clientDataJSON(ByteArray)
*/
public Step3 clientDataJSON(ByteArray clientDataJSON) {
builder.clientDataJSON(clientDataJSON);
return new Step3();
}
}

public class Step3 {
/**
* {@link AuthenticatorAssertionResponseBuilder#signature(ByteArray) signature} is a
* required parameter.
*
* @see AuthenticatorAssertionResponseBuilder#signature(ByteArray)
*/
public AuthenticatorAssertionResponseBuilder signature(ByteArray signature) {
return builder.signature(signature);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -122,12 +122,24 @@ public static class MandatoryStages {
private final AuthenticatorAttestationResponseBuilder builder =
new AuthenticatorAttestationResponseBuilder();

/**
* {@link AuthenticatorAttestationResponseBuilder#attestationObject(ByteArray)
* attestationObject} is a required parameter.
*
* @see AuthenticatorAttestationResponseBuilder#attestationObject(ByteArray)
*/
public Step2 attestationObject(ByteArray attestationObject) {
builder.attestationObject(attestationObject);
return new Step2();
}

public class Step2 {
/**
* {@link AuthenticatorAttestationResponseBuilder#clientDataJSON(ByteArray) clientDataJSON}
* is a required parameter.
*
* @see AuthenticatorAttestationResponseBuilder#clientDataJSON(ByteArray)
*/
public AuthenticatorAttestationResponseBuilder clientDataJSON(ByteArray clientDataJSON) {
return builder.clientDataJSON(clientDataJSON);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -120,19 +120,36 @@ private MandatoryStages start() {
public class MandatoryStages {
private final PublicKeyCredentialBuilder<A, B> builder;

/**
* {@link PublicKeyCredentialBuilder#id(ByteArray) id} is a required parameter.
*
* @see PublicKeyCredentialBuilder#id(ByteArray)
*/
public Step2 id(ByteArray id) {
builder.id(id);
return new Step2();
}

public class Step2 {
/**
* {@link PublicKeyCredentialBuilder#response(AuthenticatorResponse) response} is a required
* parameter.
*
* @see PublicKeyCredentialBuilder#response(AuthenticatorResponse)
*/
public Step3 response(A response) {
builder.response(response);
return new Step3();
}
}

public class Step3 {
/**
* {@link PublicKeyCredentialBuilder#clientExtensionResults(ClientExtensionOutputs)
* clientExtensionResults} is a required parameter.
*
* @see PublicKeyCredentialBuilder#clientExtensionResults(ClientExtensionOutputs)
*/
public PublicKeyCredentialBuilder<A, B> clientExtensionResults(B clientExtensionResults) {
return builder.clientExtensionResults(clientExtensionResults);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -233,30 +233,50 @@ public static class MandatoryStages {
private PublicKeyCredentialCreationOptionsBuilder builder =
new PublicKeyCredentialCreationOptionsBuilder();

/** @see PublicKeyCredentialCreationOptions#getRp() */
/**
* {@link PublicKeyCredentialCreationOptionsBuilder#rp(RelyingPartyIdentity) rp} is a required
* parameter.
*
* @see PublicKeyCredentialCreationOptionsBuilder#rp(RelyingPartyIdentity)
*/
public Step2 rp(RelyingPartyIdentity rp) {
builder.rp(rp);
return new Step2();
}

/** @see PublicKeyCredentialCreationOptions#getUser() */
public class Step2 {
/**
* {@link PublicKeyCredentialCreationOptionsBuilder#user(UserIdentity) user} is a required
* parameter.
*
* @see PublicKeyCredentialCreationOptionsBuilder#user(UserIdentity)
*/
public Step3 user(UserIdentity user) {
builder.user(user);
return new Step3();
}
}

/** @see PublicKeyCredentialCreationOptions#getChallenge() */
public class Step3 {
/**
* {@link PublicKeyCredentialCreationOptionsBuilder#challenge(ByteArray) challenge} is a
* required parameter.
*
* @see PublicKeyCredentialCreationOptionsBuilder#challenge(ByteArray)
*/
public Step4 challenge(ByteArray challenge) {
builder.challenge(challenge);
return new Step4();
}
}

/** @see PublicKeyCredentialCreationOptions#getPubKeyCredParams() */
public class Step4 {
/**
* {@link PublicKeyCredentialCreationOptionsBuilder#pubKeyCredParams(List) pubKeyCredParams}
* is a required parameter.
*
* @see PublicKeyCredentialCreationOptionsBuilder#pubKeyCredParams(List)
*/
public PublicKeyCredentialCreationOptionsBuilder pubKeyCredParams(
List<PublicKeyCredentialParameters> pubKeyCredParams) {
return builder.pubKeyCredParams(pubKeyCredParams);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -115,6 +115,11 @@ public static class MandatoryStages {
private PublicKeyCredentialDescriptorBuilder builder =
new PublicKeyCredentialDescriptorBuilder();

/**
* {@link PublicKeyCredentialDescriptorBuilder#id(ByteArray) id} is a required parameter.
*
* @see PublicKeyCredentialDescriptorBuilder#id(ByteArray)
*/
public PublicKeyCredentialDescriptorBuilder id(ByteArray id) {
return builder.id(id);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,12 @@ public static class MandatoryStages {
private PublicKeyCredentialParametersBuilder builder =
new PublicKeyCredentialParametersBuilder();

/**
* {@link PublicKeyCredentialParametersBuilder#alg(COSEAlgorithmIdentifier) alg} is a required
* parameter.
*
* @see PublicKeyCredentialParametersBuilder#alg(COSEAlgorithmIdentifier)
*/
public PublicKeyCredentialParametersBuilder alg(COSEAlgorithmIdentifier alg) {
return builder.alg(alg);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -161,6 +161,12 @@ public static class MandatoryStages {
private PublicKeyCredentialRequestOptionsBuilder builder =
new PublicKeyCredentialRequestOptionsBuilder();

/**
* {@link PublicKeyCredentialRequestOptionsBuilder#challenge(ByteArray) challenge} is a
* required parameter.
*
* @see PublicKeyCredentialRequestOptionsBuilder#challenge(ByteArray)
*/
public PublicKeyCredentialRequestOptionsBuilder challenge(ByteArray challenge) {
return builder.challenge(challenge);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -109,6 +109,7 @@ public static class MandatoryStages {
* href="https://www.w3.org/TR/2021/REC-webauthn-2-20210408/#scope">RP ID: scope</a> for
* details and examples.
*
* @see RelyingPartyIdentityBuilder#id(String)
* @see <a href="https://www.w3.org/TR/2021/REC-webauthn-2-20210408/#rp-id">RP ID</a>
* @see <a href="https://www.w3.org/TR/2021/REC-webauthn-2-20210408/#scope">RP ID: scope</a>
*/
Expand All @@ -122,6 +123,8 @@ public class Step2 {
* The human-palatable name of the Relaying Party.
*
* <p>For example: "ACME Corporation", "Wonderful Widgets, Inc." or "ОАО Примертех".
*
* @see RelyingPartyIdentityBuilder#name(String)
*/
public RelyingPartyIdentityBuilder name(String name) {
return builder.name(name);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -139,19 +139,34 @@ public static class UserIdentityBuilder {
public static class MandatoryStages {
private UserIdentityBuilder builder = new UserIdentityBuilder();

/**
* {@link UserIdentityBuilder#name(String) name} is a required parameter.
*
* @see UserIdentityBuilder#name(String)
*/
public Step2 name(String name) {
builder.name(name);
return new Step2();
}

public class Step2 {
/**
* {@link UserIdentityBuilder#displayName(String) displayName} is a required parameter.
*
* @see UserIdentityBuilder#displayName(String)
*/
public Step3 displayName(String displayName) {
builder.displayName(displayName);
return new Step3();
}
}

public class Step3 {
/**
* {@link UserIdentityBuilder#id(ByteArray) id} is a required parameter.
*
* @see UserIdentityBuilder#id(ByteArray)
*/
public UserIdentityBuilder id(ByteArray id) {
return builder.id(id);
}
Expand Down

0 comments on commit d8e2d60

Please sign in to comment.