Improve client IP resolution for Stream logs#1459

Merged

schlessera merged 23 commits intodevelopfrom add/event-source-ip

Oct 24, 2023

+75-28

Commits on Oct 16, 2023

Make the plugin responsible for resolving request IPs

kasparsd
authored and
schlessera
committed
Make the logger rely on the IP resolved by the plugin

kasparsd
authored and
schlessera
committed
Add a notice to plugin settings in case request IP can't be determined

kasparsd
authored and
schlessera
committed
Document the IP resolver logic

kasparsd
authored and
schlessera
committed
Add an example

kasparsd
authored and
schlessera
committed
Limit the notice to the Stream admin pages

kasparsd
authored and
schlessera
committed
Ensure we fail gracefully when the WP core helper is not available

kasparsd
authored and
schlessera
committed
Fix docblock syntax

kasparsd
authored and
schlessera
committed
Make it accessible

kasparsd
authored and
schlessera
committed
Add basic tests

kasparsd
authored and
schlessera
committed
Test for invalid IP

kasparsd
authored and
schlessera
committed
Use the local helper for consistency

kasparsd
authored and
schlessera
committed
Update readme.txt

kasparsd
and
schlessera
committed
Apply suggestions from code review

kasparsd
and
schlessera
committed
Apply suggestions from code review

kasparsd
and
schlessera
committed
Skip any formatting for simplicity

kasparsd
authored and
schlessera
committed
Describe the output types

kasparsd
authored and
schlessera
committed
Account for multiple IPs in the forwarded header

kasparsd
authored and
schlessera
committed
Show the notice if client IP missing

kasparsd
authored and
schlessera
committed
Don’t even attempt to use the unsafe option

kasparsd
authored and
schlessera
committed
We no longer default to a fallback

kasparsd
authored and
schlessera
committed
Add the changelog

kasparsd
authored and
schlessera
committed
Add noticeable warning regarding HTTP_* spoofing
schlessera
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Improve client IP resolution for Stream logs#1459

Commits on Oct 16, 2023

Make the plugin responsible for resolving request IPs

Make the logger rely on the IP resolved by the plugin

Add a notice to plugin settings in case request IP can't be determined

Document the IP resolver logic

Add an example

Limit the notice to the Stream admin pages

Ensure we fail gracefully when the WP core helper is not available

Fix docblock syntax

Make it accessible

Add basic tests

Test for invalid IP

Use the local helper for consistency

Update readme.txt

Apply suggestions from code review

Apply suggestions from code review

Skip any formatting for simplicity

Describe the output types

Account for multiple IPs in the forwarded header

Show the notice if client IP missing

Don’t even attempt to use the unsafe option

We no longer default to a fallback

Add the changelog

Add noticeable warning regarding HTTP_* spoofing