Improved filter option to also filter `query`

hapijs / good

hapi process monitoring

Other

525 stars 161 forks source link

Improved filter option to also filter `query` #433

Closed catalint closed 8 years ago

catalint commented 8 years ago

Filter option now filters query also

use case: access_token in query land

arb commented 8 years ago

Is that really sensitive information though? If it is, it probably shouldn't be on the query string.

catalint commented 8 years ago

It's not sensitive but it's long, repetitive, and takes up storage/screen space

Other usecase I can think of now can be to remove ?random=234234241324 or ?_utm (google analytics)

I used the "remove" filter option on my side, but left "censor" to make it easier to grasp what's happening for new users

lock[bot] commented 4 years ago

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.