Skip to content

Fix reactive cache handling in NetworkPoliciesController#summary

Arthur Evstifeev requested to merge threat-monitoring-fix-reactive-caching into master

What does this MR do?

Reactive cache flow goes through the background worker which causes arguments to the PacketFlowQuery to be stringified. This causes reactive cache worker to use a different cache key from the one used by the web process, as a result background worker will not re-calc cache. This patch converts PacketFlowQuery to use arguments that will work consistently for both flows. This patch also updates controller to send :accepted on initial caching request to improve frontend polling logic.

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Arthur Evstifeev

Merge request reports