Code review comment for lp:~maxiberta/canonical-identity-provider/secure-email-links

Revision history for this message
Daniel Manrique (roadmr) wrote :

Looks good to me; FWIW it's even more elegant than my initial thought of just clobbering the scheme with https in all cases (but I missed the fact that it would screw things up for development/testing).

review: Approve

« Back to merge proposal