)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1000000,"name":"lukegb","email":"lukegb@tvl.fyi","username":"lukegb","status":"zzz"},"change_message_id":"42047c70ea5c657e0c9bb6def16b7f6ad842e0aa","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"825ab62b_89d219f2","updated":"2021-12-07 12:57:28.000000000","message":"These aren\u0027t actually used anywhere yet, right?\n\nOTOH this looks fine - people with root on whitby will presumably be able to just see what the secrets are on whitby if they need to reencrypt it.\n\nThis is a little annoying for code-review purposes, though.","commit_id":"1d9f4d42f435366d9e2177fede06298c9eba9f40"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"12a89ca4f465799712a41dcb39f01210cb3b06f0","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"76bd3835_fda93be9","in_reply_to":"18edba4f_5dac518f","updated":"2021-12-07 22:04:26.000000000","message":"\u003e These aren\u0027t actually used anywhere yet, right?\n\nWe have a variety of secrets in use on whitby that are not version-controlled (or, in any way controlled) right now - including these.\n\nThey\u0027re not used anywhere yet *from the repo*, no. The next step is to add whitby configuration that uses the agenix module to do the decryption and placement locally on activation.\n\n\u003e that would be better implemented as a browser extension that does it locally to avoid accidental authn/authz failure\n\nI like this idea. Is there a browser-compatible age implementation? Maybe rage -\u003e wasm or something.","commit_id":"1d9f4d42f435366d9e2177fede06298c9eba9f40"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"d0480e87033e4d9d99a3aef9b8e93ea8791931d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"05178935_7d74af2c","in_reply_to":"76bd3835_fda93be9","updated":"2021-12-08 18:21:55.000000000","message":"Marking as resolved because this comment thread isn\u0027t technically related to the change itself.","commit_id":"1d9f4d42f435366d9e2177fede06298c9eba9f40"},{"author":{"_account_id":1000000,"name":"lukegb","email":"lukegb@tvl.fyi","username":"lukegb","status":"zzz"},"change_message_id":"9ac755a906d352f915408ec828094d161ec975ae","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"18edba4f_5dac518f","in_reply_to":"825ab62b_89d219f2","updated":"2021-12-07 12:58:13.000000000","message":"(maybe it would be neat to have some sort of magic Gerrit plugin that filters the age output if you\u0027re a Magic Authorised Person. but maybe that would be better implemented as a browser extension that does it locally to avoid accidental authn/authz failure)","commit_id":"1d9f4d42f435366d9e2177fede06298c9eba9f40"}]}