Remove bugsnag

Signed-off-by: David van der Spek <vanderspek.david@gmail.com>

configuration/configuration.go

@@ -62,9 +62,6 @@ type Configuration struct {
 	// Middleware lists all middlewares to be used by the registry.
 	Middleware map[string][]Middleware `yaml:"middleware,omitempty"`
 
-	// Reporting is the configuration for error reporting
-	Reporting Reporting `yaml:"reporting,omitempty"`
-
 	// HTTP contains configuration parameters for the registry's http
 	// interface.
 	HTTP struct {
@@ -627,23 +624,6 @@ type Ignore struct {
 	Actions    []string `yaml:"actions"`    // ignore action types
 }
 
-// Reporting defines error reporting methods.
-type Reporting struct {
-	// Bugsnag configures error reporting for Bugsnag (bugsnag.com).
-	Bugsnag BugsnagReporting `yaml:"bugsnag,omitempty"`
-}
-
-// BugsnagReporting configures error reporting for Bugsnag (bugsnag.com).
-type BugsnagReporting struct {
-	// APIKey is the Bugsnag api key.
-	APIKey string `yaml:"apikey,omitempty"`
-	// ReleaseStage tracks where the registry is deployed.
-	// Examples: production, staging, development
-	ReleaseStage string `yaml:"releasestage,omitempty"`
-	// Endpoint is used for specifying an enterprise Bugsnag endpoint.
-	Endpoint string `yaml:"endpoint,omitempty"`
-}
-
 // Middleware configures named middlewares to be applied at injection points.
 type Middleware struct {
 	// Name the middleware registers itself as

configuration/configuration_test.go

@@ -50,11 +50,6 @@ var configStruct = Configuration{
 			"service": "silly",
 		},
 	},
-	Reporting: Reporting{
-		Bugsnag: BugsnagReporting{
-			APIKey: "BugsnagApiKey",
-		},
-	},
 	Notifications: Notifications{
 		Endpoints: []Endpoint{
 			{
@@ -201,9 +196,6 @@ notifications:
            - application/octet-stream
         actions:
            - pull
-reporting:
-  bugsnag:
-    apikey: BugsnagApiKey
 http:
   clientcas:
     - /path/to/ca.pem
@@ -286,7 +278,6 @@ func (suite *ConfigSuite) TestParseSimple(c *check.C) {
 // a string can be parsed into a Configuration struct with no storage parameters
 func (suite *ConfigSuite) TestParseInmemory(c *check.C) {
 	suite.expectedConfig.Storage = Storage{"inmemory": Parameters{}}
-	suite.expectedConfig.Reporting = Reporting{}
 	suite.expectedConfig.Log.Fields = nil
 	suite.expectedConfig.Redis = struct {
 		Addr     string `yaml:"addr,omitempty"`
@@ -322,7 +313,6 @@ func (suite *ConfigSuite) TestParseIncomplete(c *check.C) {
 	suite.expectedConfig.Log.Fields = nil
 	suite.expectedConfig.Storage = Storage{"filesystem": Parameters{"rootdirectory": "/tmp/testroot"}}
 	suite.expectedConfig.Auth = Auth{"silly": Parameters{"realm": "silly"}}
-	suite.expectedConfig.Reporting = Reporting{}
 	suite.expectedConfig.Notifications = Notifications{}
 	suite.expectedConfig.HTTP.Headers = nil
 	suite.expectedConfig.Redis = struct {
@@ -448,20 +438,6 @@ func (suite *ConfigSuite) TestParseInvalidLoglevel(c *check.C) {
 	c.Assert(err, check.NotNil)
 }
 
-// TestParseWithDifferentEnvReporting validates that environment variables
-// properly override reporting parameters
-func (suite *ConfigSuite) TestParseWithDifferentEnvReporting(c *check.C) {
-	suite.expectedConfig.Reporting.Bugsnag.APIKey = "anotherBugsnagApiKey"
-	suite.expectedConfig.Reporting.Bugsnag.Endpoint = "localhost:8080"
-
-	os.Setenv("REGISTRY_REPORTING_BUGSNAG_APIKEY", "anotherBugsnagApiKey")
-	os.Setenv("REGISTRY_REPORTING_BUGSNAG_ENDPOINT", "localhost:8080")
-
-	config, err := Parse(bytes.NewReader([]byte(configYamlV0_1)))
-	c.Assert(err, check.IsNil)
-	c.Assert(config, check.DeepEquals, suite.expectedConfig)
-}
-
 // TestParseInvalidVersion validates that the parser will fail to parse a newer configuration
 // version than the CurrentVersion
 func (suite *ConfigSuite) TestParseInvalidVersion(c *check.C) {
@@ -475,10 +451,6 @@ func (suite *ConfigSuite) TestParseInvalidVersion(c *check.C) {
 // TestParseExtraneousVars validates that environment variables referring to
 // nonexistent variables don't cause side effects.
 func (suite *ConfigSuite) TestParseExtraneousVars(c *check.C) {
-	suite.expectedConfig.Reporting.Bugsnag.Endpoint = "localhost:8080"
-
-	// A valid environment variable
-	os.Setenv("REGISTRY_REPORTING_BUGSNAG_ENDPOINT", "localhost:8080")
 
 	// Environment variables which shouldn't set config items
 	os.Setenv("REGISTRY_DUCKS", "quack")
@@ -611,9 +583,6 @@ func copyConfig(config Configuration) *Configuration {
 	for k, v := range config.Storage.Parameters() {
 		configCopy.Storage.setParameter(k, v)
 	}
-	configCopy.Reporting = Reporting{
-		Bugsnag: BugsnagReporting{config.Reporting.Bugsnag.APIKey, config.Reporting.Bugsnag.ReleaseStage, config.Reporting.Bugsnag.Endpoint},
-	}
 
 	configCopy.Auth = Auth{config.Auth.Type(): Parameters{}}
 	for k, v := range config.Auth.Parameters() {