Skip to content

Fixed incorrect behavior of :BlockedApiPolicy #9742#9747

Merged
kcondon merged 1 commit intoIQSS:developfrom
cesarpferreira:9742-API_loadbalancer_bug
Aug 1, 2023
Merged

Fixed incorrect behavior of :BlockedApiPolicy #9742#9747
kcondon merged 1 commit intoIQSS:developfrom
cesarpferreira:9742-API_loadbalancer_bug

Conversation

@cesarpferreira
Copy link
Contributor

What this PR does / why we need it:
Documentation from our findings when solving an issue related to incorrect behavior of the policy to block API access.

Which issue(s) this PR closes:

Closes #9742

Special notes for your reviewer:

Suggestions on how to test this:

Does this PR introduce a user interface change? If mockups are available, please link/include them here:
No.

Is there a release notes update needed for this change?:
No.

Additional documentation:

Copy link
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

@donsizemore donsizemore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this specific to nginx? if so, should we make a note of that?

@pdurbin
Copy link
Member

pdurbin commented Jul 31, 2023

@cesarpferreira thanks for the pull request! Please note the question above by @donsizemore. Thanks!

@pdurbin pdurbin self-assigned this Jul 31, 2023
@cesarpferreira
Copy link
Contributor Author

is this specific to nginx? if so, should we make a note of that?

@donsizemore that is a good question. We were doing some tests with haproxy in one installation, but it had the host IP address so it was working correctly. I can try to change it to localhost and check if this happens

@cesarpferreira
Copy link
Contributor Author

is this specific to nginx? if so, should we make a note of that?

@donsizemore that is a good question. We were doing some tests with haproxy in one installation, but it had the host IP address so it was working correctly. I can try to change it to localhost and check if this happens

Just confirmed, it also happens with haproxy and this situation is most common when you only have one server behind the reverse proxy service and it is easier to set up the backend as localhost.

@pdurbin
Copy link
Member

pdurbin commented Aug 1, 2023

Does that mean we're ok with the pull request as-is? Or should @cesarpferreira add an additional note? (@cesarpferreira please feel free to add more if you like!)

Copy link
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We just talked about this at standup. @donsizemore and I are happy with it as-is so I'm moving it to QA.

@kcondon kcondon self-assigned this Aug 1, 2023
@kcondon kcondon merged commit 54fd71e into IQSS:develop Aug 1, 2023
@pdurbin pdurbin added this to the 5.14 milestone Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect behavior of :BlockedApiPolicy when set to localhost-only and when using a load balancer

4 participants