Add request protector app with cross-domain cookies filtering

Sohbet 4 İşlemeler 35 Değiştirilen Dosyalar 17

@jonathan please review

Waiting on Jenkins build: https://jenkins.bubblev.org/job/bubble-dev-kris/252/
but not sure how it will go as current master build is failing.

jonathan 4 yıl önce kris tarafından atandı

Looks good, but what would the regex be if the intent is “delete any cookies NOT belonging to the current domain, or one of its subdomains”?

It seems like we would need some more logic -- if we add these 2 things I think it would work:

The regex can include {{fqdn}} which will be replaced (in mitm-land) with the (regex-escaped) FQDN for the current request
In addition to regex and replacement, add mode which can be match or no_match. Or, alternatively, see if the “regex negation” syntax works in Python’s regex implementation, see: https://stackoverflow.com/questions/2637675/how-to-negate-the-whole-regex ). If regex negation works, that may be easier.

Thoughts?

will do. Separate PR? (easier review)
(?!...) is supported in python3’s re

will do. Separate PR? (easier review)

No, let’s do it in this one. And test it :) this will be very cool when it works!

(?!...) is supported in python3’s re

That’s good news. Let’s do that.

👍 1

kris başlık ~~Add request protector app with cross-domain cookies filtering~~ iken WIP: (testing after fqdn part) Add request protector app with cross-domain cookies filtering olarak değiştirildi 4 yıl önce

kris başlık ~~WIP: (testing after fqdn part) Add request protector app with cross-domain cookies filtering~~ iken Add request protector app with cross-domain cookies filtering olarak değiştirildi 4 yıl önce

kris başlık ~~Add request protector app with cross-domain cookies filtering~~ iken WIP: (waiting on fresh Jenkins build) Add request protector app with cross-domain cookies filtering olarak değiştirildi 4 yıl önce

kris başlık ~~WIP: (waiting on fresh Jenkins build) Add request protector app with cross-domain cookies filtering~~ iken Add request protector app with cross-domain cookies filtering olarak değiştirildi 4 yıl önce

Passing Jenkins build: https://jenkins.bubblev.org/job/bubble-dev-kris/260/

jonathan 4 yıl önce değişiklik istedi

minor changes requested.

bubble-server/src/main/resources/models/defaults/bubblePlan.json

												
			@@ -38,2 +38,3 @@
		
			        {"app": "UserBlocker"},
		
			        {"app": "TlsPassthru"}
		
			        {"app": "TlsPassthru"},
		
			        {"app": "RequestProtector"}

jonathan 4 yıl önce yorum yaptı

Let’s add RequestProtector app to all plans.

bubble-server/src/main/resources/packer/roles/mitmproxy/files/bubble_api.py

jonathan 4 yıl önce işlemesinde bu konuyu işaret etti

jonathan 4 yıl önce değişiklik isteğini kapattı

jonathan kris/request_protector_app dalı silindi 4 yıl önce

Gözden Geçirenler

jonathan 4 yıl önce değişiklik istedi

Değişiklik isteği 237e90d126 olarak birleştirildi.

Bu konuşmaya katılmak için oturum aç.

jonathan

Etiket Yok

Kilometre Taşı Yok

jonathan

2 Katılımcı

Bildirimler

Bitiş Tarihi

Bitiş tarihi atanmadı.

Bağımlılıklar

Bu çekme isteği henüz bir bağımlılık içermiyor.

@@ -452,0 +487,4 @@
 def _replace_in_headers(headers: nheaders.Headers, modifiers_dict: dict) -> int:
     """
     Taken from original mitmproxy's Header class implementation with sligh change to allow replacement with empty string

#58 Add request protector app with cross-domain cookies filtering

Gözden Geçirenler