octet-stream.net
targeting
master
from
octet-stream.net/core:
thombles/error-logged-in
Closes https://tangled.org/tangled.org/core/issues/272
Call sites are updated to provide user, which was usually present anyway. There were a couple of exceptions where I passed nil in functions that were creating an atom feed, since it seemed unlikely to be authenticated.
Signed-off-by: Thomas Karpiniec <tkarpiniec@icloud.com>
Aye you're right, Context would be cleaner. It's only a cosmetic improvement so I'd suggest we just close and try again later when the refactoring is done. :) I'll leave a note on the issue.
tangled.org
I fill guilty about this... There was PR #692 with same change which I didn't merge because I thought using
ctx.Valuewould be more ideal approach. I still think so, but now I think it's fine to accept as a temporary solution as I'm rewriting all existing handlers anyway (PR #800).