larryh/corso
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

call repo connect in connect S3 cmd (#4383)

Browse Source 
<!-- PR description-->

Call repo.connect() in S3 connect command. 
Was this missed as part of - https://github.com/alcionai/corso/pull/4343/files or am I misunderstanding something. 

#### Does this PR need a docs update or release note?

- [ ] ✅ Yes, it's included

#### Type of change

<!--- Please check the type of change your PR introduces: --->
- [ ] 🐛 Bugfix

#### Issue(s)

<!-- Can reference multiple issues. Use one of the following "magic words" - "closes, fixes" to auto-close the Github issue. -->
* #<issue>

#### Test Plan

<!-- How will this be tested prior to merging.-->
- [ ] 💪 Manua
This commit is contained in:
neha_gupta 2023-09-27 22:07:15 +05:30 committed by GitHub
parent 91c0709d09
commit 38ef3b6ef6
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/cli/repo/s3.go
View File

@ -222,6 +222,10 @@ func connectS3Cmd(cmd *cobra.Command, args []string) error {
opts, opts,
repoID) repoID)
if err != nil { if err != nil {
return Only(ctx, clues.Wrap(err, "Failed to create a repository controller"))
}
if err := r.Connect(ctx); err != nil {
return Only(ctx, clues.Wrap(err, "Failed to connect to the S3 repository")) return Only(ctx, clues.Wrap(err, "Failed to connect to the S3 repository"))
} }