search

litespeedtech / openlitespeed

Our high-performance, lightweight, open source HTTP server
https://openlitespeed.org
GNU General Public License v3.0
1.16k stars 189 forks source link

docs: config/logs/visitorip/ #417

Closed epreston closed 3 weeks ago

epreston commented 1 month ago

Could you direct me where I can contribute a documentation update ?

I would like to update the following page:

https://docs.openlitespeed.org/config/logs/visitorip/

The updates are:

References:

https://www.cloudflare.com/en-au/ips/

epreston commented 1 month ago

Two more updates:

104.16.0.0/12 removed from ips-v4 104.16.0.0/13 added to ips-v4

lslisa commented 3 weeks ago

Thanks for the report, @epreston. The listed IPs in the doc are meant to be simply an example to show how it is done, not an exhaustive list of Cloudflare IPs. There is a link right above the list that will get you Cloudflare's current IP list.

I will update the documentation to be more clear about that.

epreston commented 3 weeks ago

@lslisa do you know the repo and location to submit a PR to ?

Just want to update the values, already clear enough that it's an example, not looking for the reference since I've included it above.

My main issue is including an old range that cloudflare now uses for a different purpose in the trusted set.

lslisa commented 3 weeks ago

The repo isn't public. I'll make those changes. Or maybe I will change the example to use nonsense numbers (like 123.45.67.89/00) I plan to look at it later today. Thank you for the additional context.

epreston commented 3 weeks ago

@litespeedtech can I get someone else to help with this ?

lslisa commented 3 weeks ago

@epreston I am the director of content at LiteSpeed, and this includes Documentation. What additional help did you require?

lslisa commented 3 weeks ago

@epreston I've updated the doc. The CF IPs have been replaced with example IPs. Thank you for bringing this to our attention. Let me know if you have any further issues with that (or other) documentation.

epreston commented 3 weeks ago

@lslisa No thanks. Given the results I'll go through sales or customer support if I notice any issues.