Repository: theupdateframework/go-tuf (go)

Issue:
I increase the default `MaxRootRotations` value for the updater config.
The current default value is 32, which is quite small. For some scenarios where the repository is large, a refresh from the first root.json may fail. 
Note that clients *should* cache the metadata, to avoid large number of rotations which is inefficient, and clients *should* embed a "fresh" version of the root to avoid unnecessary rotations, but in some scenarios it's unavoidable, such as when verifying the complete chain of root files. 

Pull request:
Increase the default value for MaxRootRotations
The default value of 32 is quite small, it may break certain clients that are trying to do complete tuf refresh on a large TUF repo.

Closes #644 

Patch:
diff --git a/metadata/config/config.go b/metadata/config/config.go
index 09c9b46..cd8cc98 100644
--- a/metadata/config/config.go
+++ b/metadata/config/config.go
@@ -57,7 +57,7 @@ func New(remoteURL string, rootBytes []byte) (*UpdaterConfig, error) {
 
 	return &UpdaterConfig{
 		// TUF configuration
-		MaxRootRotations:   32,
+		MaxRootRotations:   256,
 		MaxDelegations:     32,
 		RootMaxLength:      512000,  // bytes
 		TimestampMaxLength: 16384,   // bytes


Test patch:
diff --git a/metadata/config/config_test.go b/metadata/config/config_test.go
index be35a87..a5d4874 100644
--- a/metadata/config/config_test.go
+++ b/metadata/config/config_test.go
@@ -43,7 +43,7 @@ func TestNewUpdaterConfig(t *testing.T) {
 			remoteURL: "somepath",
 			rootBytes: []byte("somerootbytes"),
 			config: &UpdaterConfig{
-				MaxRootRotations:      32,
+				MaxRootRotations:      256,
 				MaxDelegations:        32,
 				RootMaxLength:         512000,
 				TimestampMaxLength:    16384,