remove access token length check (#412)

launcher/config.go

@@ -287,11 +287,6 @@ func validateConfiguration(c Config) error {
 		}
 	}
 
-	// TODO(@tobert) will probably break on some providers but seems fine for my use cases right now
-	if accessTokenLen > 0 && (accessTokenLen != 32 && accessTokenLen != 84 && accessTokenLen != 104 && accessToken(c) != "developer") {
-		return fmt.Errorf("invalid configuration: access token length incorrect. Ensure token is set correctly")
-	}
-
 	return nil
 }
 

launcher/config_test.go

@@ -35,7 +35,6 @@ import (
 )
 
 const (
-	expectedAccessTokenLengthError  = "invalid configuration: access token length incorrect. Ensure token is set correctly"
 	expectedAccessTokenMissingError = "invalid configuration: access token missing, must be set when reporting to ingest.lightstep.com"
 	expectedTracingDisabledMessage  = "tracing is disabled by configuration: no endpoint set"
 	expectedMetricsDisabledMessage  = "metrics are disabled by configuration: no endpoint set"
@@ -207,34 +206,6 @@ func (suite *testSuite) TestInvalidMetricDefaultAccessToken() {
 	)
 }
 
-func (suite *testSuite) testInvalidAccessToken(opts ...Option) {
-	lsOtel := ConfigureOpentelemetry(
-		append(opts,
-			WithLogger(&suite.testLogger),
-			WithServiceName("test-service"),
-		)...,
-	)
-	defer lsOtel.Shutdown()
-
-	suite.requireLogContains(expectedAccessTokenLengthError)
-}
-
-func (suite *testSuite) TestInvalidTraceAccessTokenLength() {
-	suite.testInvalidAccessToken(
-		append(suite.insecureTraceEndpointOptions(),
-			WithAccessToken("1234"),
-		)...,
-	)
-}
-
-func (suite *testSuite) TestInvalidMetricAccessTokenLength() {
-	suite.testInvalidAccessToken(
-		append(suite.bothInsecureEndpointOptions(),
-			WithAccessToken("1234"),
-		)...,
-	)
-}
-
 func (suite *testSuite) testEndpointDisabled(expected string, opts ...Option) {
 	lsOtel := ConfigureOpentelemetry(
 		append(opts,