update policy name convention enforcement to allow patterns

apigee / apigeelint

Static code analysis for Apigee proxy bundles to encourage API developers to use best practices and avoid anti-patterns.

Apache License 2.0

92 stars 71 forks source link

Closed DinoChiesa closed 2 years ago

DinoChiesa commented 2 years ago

eg, name of CORS policy can be "CORS", with no suffix. Similarly with Quota and HMAC policies.
new tests for same.
also added in metadata for JWT policy types.

ssvaidyanathan commented 2 years ago

@DinoChiesa - Can we also include GraphQL?

DinoChiesa commented 2 years ago

Yes - let me make that change.

kurtkanaskie commented 2 years ago

Graph QL could be GQ-parse or GQ-validate or GQ-parse-validate, those are the different uses.

DinoChiesa commented 2 years ago

For prefixes, I used only "graphql" , not GQ. Do we want GQ? also the policy can be just "GraphQL", also. IE, either GraphQL-suffix or GraphQL.

ssvaidyanathan commented 2 years ago

For prefixes, I used only "graphql" , not GQ. Do we want GQ? also the policy can be just "GraphQL", also. IE, either GraphQL-suffix or GraphQL.

I am fine with the options

kurtkanaskie commented 2 years ago

I prefer to use 2 letter prefixes for all my policies. I would like to have "GQ" as a valid prefix.

DinoChiesa commented 2 years ago

Kurt, I will add "GQ" as a prefix for the GraphQL policy.