Add request protector app with cross-domain cookies filtering

Konverzace 4 Revize 35 Změněné soubory 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 byl přiřazen kris před 4 roky

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 změnil(a) název z ~~Add request protector app with cross-domain cookies filtering~~ na WIP: (testing after fqdn part) Add request protector app with cross-domain cookies filtering před 4 roky

kris změnil(a) název z ~~WIP: (testing after fqdn part) Add request protector app with cross-domain cookies filtering~~ na Add request protector app with cross-domain cookies filtering před 4 roky

kris změnil(a) název z ~~Add request protector app with cross-domain cookies filtering~~ na WIP: (waiting on fresh Jenkins build) Add request protector app with cross-domain cookies filtering před 4 roky

kris změnil(a) název z ~~WIP: (waiting on fresh Jenkins build) Add request protector app with cross-domain cookies filtering~~ na Add request protector app with cross-domain cookies filtering před 4 roky

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

jonathan požadované změny před 4 roky

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 okomentoval před 4 roky

Let’s add RequestProtector app to all plans.

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

jonathan odkázal na tento úkol z revize před 4 roky

jonathan closed this pull request před 4 roky

jonathan odstranil(a) větev kris/request_protector_app před 4 roky

Posuzovatelé

jonathan požadované změny před 4 roky

Požadavek na natažení byl sloučen jako 237e90d126.

Přihlaste se pro zapojení do konverzace.

jonathan

Bez štítku

Bez milníku

jonathan

2 účastníků

Oznámení

Termín dokončení

Žádný termín dokončení.

Závislosti

Tento požadavek na natažení momentálně nemá žádné závislosti.

@@ -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

Posuzovatelé