search

guardian / grid

The Guardian’s image management system
https://www.theguardian.com/info/developer-blog/2015/aug/12/open-sourcing-grid-image-service
Apache License 2.0
1.44k stars 120 forks source link

override UserPrincipal.toString to avoid logging info that we shouldn't be logging #4211

Closed andrew-nowak closed 9 months ago

andrew-nowak commented 9 months ago

What does this change?

there's a field that we shouldn't have been logging but we have for a while oops.

override this toString to not contain that field so we stop logging it

github-actions[bot] commented 9 months ago

Deploy build 12173 to TEST

All deployment options - [Deploy build 12173 to TEST](https://riffraff.gutools.co.uk/deployment/deployAgain?project=media-service%3A%3Agrid%3A%3Aall&build=12173&stage=TEST&updateStrategy=MostlyHarmless&action=deploy) - [Deploy parts of build 12173 to TEST by previewing it first](https://riffraff.gutools.co.uk/preview/yaml?project=media-service%3A%3Agrid%3A%3Aall&build=12173&stage=TEST&updateStrategy=MostlyHarmless)

From guardian/actions-riff-raff.

prout-bot commented 9 months ago

Seen on auth, usage, image-loader, metadata-editor, thrall, leases, cropper, collections, media-api, kahuna (merged by @andrew-nowak 8 minutes and 52 seconds ago) Please check your changes!