Purge requests: Parent field
This rule enforces that all Purge
methods have a string parent
field in the request message, as mandated in AIP-165, unless the resource is
top-level.
Details
This rule looks at any message matching Purge*Request
and complains if
either the parent
field is missing, or if it has any type other than string
.
This rule is skipped for top-level resources, since they have no parent.
Examples
Incorrect code for this rule:
// Incorrect.
// Should include a `string parent` field.
message PurgeBooksRequest {
string filter = 2 [(google.api.field_behavior) = REQUIRED];
bool force = 3;
}
// Incorrect.
message PurgeBooksRequest {
// Field type should be `string`.
bytes parent = 1 [
(google.api.field_behavior) = REQUIRED,
(google.api.resource_reference).child_type = "library.googleapis.com/Book"
];
string filter = 2 [(google.api.field_behavior) = REQUIRED];
bool force = 3;
}
Correct code for this rule:
// Correct.
message PurgeBooksRequest {
string parent = 1 [
(google.api.field_behavior) = REQUIRED,
(google.api.resource_reference).child_type = "library.googleapis.com/Book"
];
string filter = 2 [(google.api.field_behavior) = REQUIRED];
bool force = 3;
}
Disabling
If you need to violate this rule, use a leading comment above the message (if
the parent
field is missing) or above the field (if it is the wrong type).
Remember to also include an aip.dev/not-precedent comment explaining why.
message PurgeBooksRequest {
// (-- api-linter: core::0165::request-parent-field=disabled
// aip.dev/not-precedent: We need to do this because reasons. --)
bytes parent = 1 [
(google.api.field_behavior) = REQUIRED,
(google.api.resource_reference).child_type = "library.googleapis.com/Book"
];
string filter = 2 [(google.api.field_behavior) = REQUIRED];
bool force = 3;
}
If you need to violate this rule for an entire file, place the comment at the top of the file.