Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

Document Envoy error messages when traffic is blocked via an SMI policy #1085

Closed
phillipgibson opened this issue Jul 14, 2020 · 4 comments
Closed
Labels
size/M 7 days (~1.5 week)

Comments

@phillipgibson
Copy link
Contributor

No description provided.

@phillipgibson phillipgibson added P2 open source launch Activities related to project open sourcing labels Jul 14, 2020
@draychev
Copy link
Contributor

SMI policies are allow-by-default

When a route is not allowed in SMI - Envoy on the client 404 because the rotue does not exist on the client Envoy
When the traffic is allowed for some reason to reach the destination, certificates would be blocked and error then would be 503 - this could happen in some cases w/ egress.
When in allow-all mode this would be 200 ok

How do we make sure that Envoy on the client side returns appropriate message. What's an appropriate message 403?

@shashankram
Copy link
Member

SMI policies are allow-by-default

When a route is not allowed in SMI - Envoy on the client 404 because the rotue does not exist on the client Envoy
When the traffic is allowed for some reason to reach the destination, certificates would be blocked and error then would be 503 - this could happen in some cases w/ egress.
When in allow-all mode this would be 200 ok

How do we make sure that Envoy on the client side returns appropriate message. What's an appropriate message 403?

The behavior described above was changed by PR #1182: whether egress is enabled or not the client will receive a 404 if the destination can't be reached.

@bridgetkromhout bridgetkromhout added Improvement / Feature Request and removed open source launch Activities related to project open sourcing labels Aug 4, 2020
@draychev draychev changed the title Proper error messaging for SMI blocked traffic Document Envoy error messages when traffic is blocked via an SMI policy Sep 15, 2020
@draychev draychev removed the P2 label Oct 26, 2020
@draychev
Copy link
Contributor

We need to compose this document after #705 is finished - it will change these error messages.

@shashankram
Copy link
Member

Different error codes emitted by the control plane are now well documented at https://docs.openservicemesh.io/docs/guides/troubleshooting/control_plane_error_codes/.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/M 7 days (~1.5 week)
Projects
None yet
Development

No branches or pull requests

4 participants