Giter Site home page Giter Site logo

Comments (11)

kyessenov avatar kyessenov commented on May 25, 2024 1

Are you saying that sidecar Envoy strips XFF? It seems that sidecar operates in the transparent mode, meaning the header is preserved https://lyft.github.io/envoy/docs/configuration/http_conn_man/headers.html#id9

from old_issues_repo.

rshriram avatar rshriram commented on May 25, 2024 1

from old_issues_repo.

rshriram avatar rshriram commented on May 25, 2024

from old_issues_repo.

christopherL91 avatar christopherL91 commented on May 25, 2024

The problem occurs on gke since google is setting the client ip in a X-Forwarded-For header which then needs to be passed down. For now I have an nginx container before istio that does ssl-termination and passes the XFF headers to istio, but the istio proxy is filtering them out before the request ends up in the server. I wasn't able to get istio go pick up the XFF headers on it's own. Maybe there's a config that needs to tweaked?

from old_issues_repo.

christopherL91 avatar christopherL91 commented on May 25, 2024

btw, is it an acceptable solution to set a container before istio, or are you meant to terminate inside istio? I'm running istio-auth

from old_issues_repo.

rshriram avatar rshriram commented on May 25, 2024

from old_issues_repo.

christopherL91 avatar christopherL91 commented on May 25, 2024

Yes I would like to have the XFF headers inside the application. How do you fix that?

from old_issues_repo.

rshriram avatar rshriram commented on May 25, 2024

from old_issues_repo.

christopherL91 avatar christopherL91 commented on May 25, 2024

Tested without any difference. The istio proxy seems to be working in "isolated mode" and will not trust any of the external XFF headers. Will you be able to flip the "use_remote_address" flag in an upcoming release?

from old_issues_repo.

ldemailly avatar ldemailly commented on May 25, 2024

@christopherL91 does 0.1.6 work for you now ?

from old_issues_repo.

ldemailly avatar ldemailly commented on May 25, 2024

closing assuming this is solved, please reopen if not

from old_issues_repo.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.