diff --git a/app/diagnostics.go b/app/diagnostics.go index 62dcc0b644..713d8aa264 100644 --- a/app/diagnostics.go +++ b/app/diagnostics.go @@ -290,6 +290,7 @@ func trackConfig() { "amazon_s3_ssl": *utils.Cfg.FileSettings.AmazonS3SSL, "amazon_s3_sse": *utils.Cfg.FileSettings.AmazonS3SSE, "amazon_s3_signv2": *utils.Cfg.FileSettings.AmazonS3SignV2, + "amazon_s3_trace": *utils.Cfg.FileSettings.AmazonS3Trace, "max_file_size": *utils.Cfg.FileSettings.MaxFileSize, "enable_file_attachments": *utils.Cfg.FileSettings.EnableFileAttachments, "enable_mobile_upload": *utils.Cfg.FileSettings.EnableMobileUpload, diff --git a/config/default.json b/config/default.json index 59bcf8bc69..50bf7b32b0 100644 --- a/config/default.json +++ b/config/default.json @@ -131,7 +131,8 @@ "AmazonS3Endpoint": "s3.amazonaws.com", "AmazonS3SSL": true, "AmazonS3SignV2": false, - "AmazonS3SSE": false + "AmazonS3SSE": false, + "AmazonS3Trace": false }, "EmailSettings": { "EnableSignUpWithEmail": true, diff --git a/model/config.go b/model/config.go index 85181edd97..311a43261b 100644 --- a/model/config.go +++ b/model/config.go @@ -272,6 +272,7 @@ type FileSettings struct { AmazonS3SSL *bool AmazonS3SignV2 *bool AmazonS3SSE *bool + AmazonS3Trace *bool } type EmailSettings struct { @@ -607,6 +608,11 @@ func (o *Config) SetDefaults() { *o.FileSettings.AmazonS3SSE = false // Not Encrypted by default. } + if o.FileSettings.AmazonS3Trace == nil { + o.FileSettings.AmazonS3Trace = new(bool) + *o.FileSettings.AmazonS3Trace = false + } + if o.FileSettings.EnableFileAttachments == nil { o.FileSettings.EnableFileAttachments = new(bool) *o.FileSettings.EnableFileAttachments = true diff --git a/utils/file.go b/utils/file.go index 0ba132b098..19fa335c41 100644 --- a/utils/file.go +++ b/utils/file.go @@ -33,7 +33,17 @@ func s3New(endpoint, accessKey, secretKey string, secure bool, signV2 bool, regi } else { creds = credentials.NewStatic(accessKey, secretKey, "", credentials.SignatureV4) } - return s3.NewWithCredentials(endpoint, creds, secure, region) + + s3Clnt, err := s3.NewWithCredentials(endpoint, creds, secure, region) + if err != nil { + return nil, err + } + + if *Cfg.FileSettings.AmazonS3Trace { + s3Clnt.TraceOn(os.Stdout) + } + + return s3Clnt, nil } func TestFileConnection() *model.AppError { diff --git a/webapp/components/admin_console/storage_settings.jsx b/webapp/components/admin_console/storage_settings.jsx index b591758b18..4b20a8b934 100644 --- a/webapp/components/admin_console/storage_settings.jsx +++ b/webapp/components/admin_console/storage_settings.jsx @@ -37,6 +37,7 @@ export default class StorageSettings extends AdminSettings { config.FileSettings.AmazonS3Endpoint = this.state.amazonS3Endpoint; config.FileSettings.AmazonS3SSL = this.state.amazonS3SSL; config.FileSettings.AmazonS3SSE = this.state.amazonS3SSE; + config.FileSettings.AmazonS3Trace = this.state.amazonS3Trace; return config; } @@ -54,7 +55,8 @@ export default class StorageSettings extends AdminSettings { amazonS3Bucket: config.FileSettings.AmazonS3Bucket, amazonS3Endpoint: config.FileSettings.AmazonS3Endpoint, amazonS3SSL: config.FileSettings.AmazonS3SSL, - amazonS3SSE: config.FileSettings.AmazonS3SSE + amazonS3SSE: config.FileSettings.AmazonS3SSE, + amazonS3Trace: config.FileSettings.AmazonS3Trace }; } @@ -125,7 +127,6 @@ export default class StorageSettings extends AdminSettings { defaultMessage='Enable Server-Side Encryption for Amazon S3:' /> } - placeholder={Utils.localizeMessage('admin.image.amazonS3SSEExample', 'Ex "false"')} helpText={ } - placeholder={Utils.localizeMessage('admin.image.amazonS3SSLExample', 'Ex "true"')} helpText={ {amazonSSEComp} + + } + helpText={ + + } + value={this.state.amazonS3Trace} + onChange={this.handleChange} + disabled={this.state.driverName !== DRIVER_S3} + /> documentation to learn more.", - "admin.image.amazonS3SSEExample": "E.g.: \"false\"", "admin.image.amazonS3SSETitle": "Enable Server-Side Encryption for Amazon S3:", "admin.image.amazonS3SSLDescription": "When false, allow insecure connections to Amazon S3. Defaults to secure connections only.", - "admin.image.amazonS3SSLExample": "E.g.: \"true\"", "admin.image.amazonS3SSLTitle": "Enable Secure Amazon S3 Connections:", "admin.image.amazonS3SecretDescription": "Obtain this credential from your Amazon EC2 administrator.", "admin.image.amazonS3SecretExample": "E.g.: \"jcuS8PuvcpGhpgHhlcpT1Mx42pnqMxQY\"", "admin.image.amazonS3SecretTitle": "Amazon S3 Secret Access Key:", + "admin.image.amazonS3TraceDescription": "(Development Mode) When true, log additional debugging information to the system logs.", + "admin.image.amazonS3TraceTitle": "Enable Amazon S3 Debugging:", "admin.image.localDescription": "Directory to which files and images are written. If blank, defaults to ./data/.", "admin.image.localExample": "E.g.: \"./data/\"", "admin.image.localTitle": "Local Storage Directory:",