Reviewable status: 0 of 3 files reviewed, 2 unresolved discussions (waiting on @scrye)

a discussion (no related file):
Just make sure to edit the commit message when merging

go/lib/infra/modules/trust/v2/handlers.go, line 109 at r2 (raw file):

	}
	logger := log.FromCtx(h.request.Context())
	if h.request.Message == nil {

Wait, how does the message being nil matter?
That was fixed in a previous PR.

go/lib/infra/modules/trust/v2/handlers.go, line 113 at r2 (raw file):

		return infra.MetricsErrInternal
	}
	// XXX(scrye): In case a TRC update will invalidate the local certificate

Do we actually have an issue to track this?
We should add this to next sprint.

Reviewable status: 0 of 3 files reviewed, 3 unresolved discussions (waiting on @oncilla and @scrye)

a discussion (no related file):

Will do 👍

go/lib/infra/modules/trust/v2/handlers.go, line 109 at r2 (raw file):

Done.

go/lib/infra/modules/trust/v2/handlers.go, line 113 at r2 (raw file):

Added https://github.com/Anapaya/scion/issues/1685.

Reviewable status: 0 of 3 files reviewed, 1 unresolved discussion (waiting on @scrye)

Reviewed 2 of 3 files at r2, 1 of 1 files at r3.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @scrye)

Reviewable status: complete! all files reviewed, all discussions resolved