Corrects condition used to substitute custom authorizers configuration

konpyutaika / nifikop

The NiFiKop NiFi Kubernetes operator makes it easy to run Apache NiFi on Kubernetes. Apache NiFI is a free, open-source solution that support powerful and scalable directed graphs of data routing, transformation, and system mediation logic.

https://konpyutaika.github.io/nifikop/

Apache License 2.0

126 stars 44 forks source link

Corrects condition used to substitute custom authorizers configuration #223

Closed mh013370 closed 1 year ago

mh013370 commented 1 year ago

Q	A
Bug fix?	yes
New feature?	no
API breaks?	no
Deprecations?	no
Related tickets	fixes #221
License	Apache 2.0

What's in this PR?

Fixes issue where nifikop uses an incorrect condition to determine whether or not to substitute a custom authorizers template.

Checklist

[ ] Implementation tested
[x] Error handling code meets the guideline
[x] Logging code meets the guideline
[x] User guide and development docs updated (if needed)
[x] Append changelog with changes

juldrixx commented 1 year ago

LGTM, @erdrix may be there was a reason to do it?

mh013370 commented 1 year ago

LGTM, @erdrix may be there was a reason to do it?

This was my mistake when i added this feature. The context is explained in #221. This fixes a very minor issue, but it's 100% a bug.