Purge methods: HTTP URI parent variable
This rule enforces that all Purge RPCs map the parent field to the HTTP
URI, as mandated in AIP-165.
Details
This rule looks at any message beginning with Purge, and complains
if the parent variable is not included in the URI. It does check additional
bindings if they are present.
Examples
Incorrect code for this rule:
// Incorrect.
rpc PurgeBooks(PurgeBooksRequest) returns (google.longrunning.Operation) {
option (google.api.http) = {
post: "/v1/publishers/*/books:purge" // The `parent` field should be extracted.
body: "*"
};
option (google.longrunning.operation_info) = {
response_type: "PurgeBooksResponse"
metadata_type: "PurgeBooksMetadata"
};
}
Correct code for this rule:
// Correct.
rpc PurgeBooks(PurgeBooksRequest) returns (google.longrunning.Operation) {
option (google.api.http) = {
post: "/v1/{parent=publishers/*}/books:purge"
body: "*"
};
option (google.longrunning.operation_info) = {
response_type: "PurgeBooksResponse"
metadata_type: "PurgeBooksMetadata"
};
}
Disabling
If you need to violate this rule, use a leading comment above the method. Remember to also include an aip.dev/not-precedent comment explaining why.
// (-- api-linter: core::0165::http-parent-variable=disabled
// aip.dev/not-precedent: We need to do this because reasons. --)
rpc PurgeBooks(PurgeBooksRequest) returns (google.longrunning.Operation) {
option (google.api.http) = {
post: "/v1/publishers/*/books:purge"
body: "*""
};
option (google.longrunning.operation_info) = {
response_type: "PurgeBooksResponse"
metadata_type: "PurgeBooksMetadata"
};
}
If you need to violate this rule for an entire file, place the comment at the top of the file.
View on GitHub