Don't set the artifact metadata before the inspector approves it, or it may be approved by a different inspector and retain the metadata set by the non-approving inspector. In this case, the artifact was approved by the Charmcraft inspector, and its metadata was rewritten by the go module inspector before marking it as rejected. Also remove external URLs from the list of valid URLs for go modules in git repositories, and remove unnecessary explicit unknown opinions from the session report. Additional testing to be added in a separate PR when a disambiguation strategy for git repositories is defined. - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----