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
91 stars 71 forks source link

PO035 is enforcing {SyncIntervalInSecond, SyncMessageCount} as mutually exclusive #443

Closed DinoChiesa closed 3 months ago

DinoChiesa commented 3 months ago

The documentation for AsynchonousConfiguration states that people can use EITHER the SyncIntervalInSeconds or the SyncMessageCount. And that they are mutually exclusive. The doc does not state what happens if a configuration uses both.

link

quota-async-mutually-exclusive

But I think this may be incorrect. I think it is valid to use both elements. And when you do, it means: "Sync every N messages, or... M seconds, whichever comes first."

If this is so, then the PO035 plugin needs to tolerate both elements.

DinoChiesa commented 3 months ago

This is fixed by #445