Code Monkey home page Code Monkey logo

Comments (7)

jarlandre avatar jarlandre commented on August 15, 2024

Just tell if its interesting that i make a PR into this repo from this MaindeckAS@a8425b3

from limiter.

novln avatar novln commented on August 15, 2024

Hello,

As described here (https://github.com/ulule/limiter#limiter-behind-a-reverse-proxy) you should use what is best for your network topology. That's why it's disabled by default.

But word of caution, if you use the rightmost IP in the header you could rate limit your load balancer or CDN.

Just tell if its interesting that i make a PR into this repo [...]

No thank you, I plan to make a customizable system for the next major version.
I just need to find the motivation for this, I haven't been active recently.

from limiter.

novln avatar novln commented on August 15, 2024

But thank you for mentioning your fork, I may use it when (or if) I'll implement a rightmost ip parser.

from limiter.

jarlandre avatar jarlandre commented on August 15, 2024

But word of caution, if you use the rightmost IP in the header you could rate limit your load balancer or CDN.

No, because aws alb appends client ip to XFF header. I dont know from where you have this idea, but its not valid in aws :) but no stress

from limiter.

jarlandre avatar jarlandre commented on August 15, 2024

btw the most funny part is i might not need to use this fix at all. Because i made a middleware that fixes XFF header to rightmost. So i can use all kinds of libraries like this without caring too much

from limiter.

novln avatar novln commented on August 15, 2024

No, because aws alb appends client ip to XFF header. I dont know from where you have this idea, but its not valid in aws :)

As said before, that depend of your network topology: not everyone use AWS.
You could have an nginx with a public ip and then other nginx inside a VLAN, private network or an hypervisor that routes that to the correct VM / containers.

from limiter.

jarlandre avatar jarlandre commented on August 15, 2024

Yep, you are right :) Ill close this now, but feel free to use as inspiration, or not. I am currently using another solution for this problem.

from limiter.

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.