Skip to content

Conversation

@PVince81
Copy link
Member

Summary

  • Extend the external storage configuration parameters definition to allow to specify a default value
  • Enable SSL by default in external storage configurations (at creation time)

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Tests (unit, integration, api and/or acceptance) are included
  • Screenshots before/after for front-end changes
  • Documentation (manuals or wiki) has been updated or is not required => seems there is no doc at all about writing ext storages, so skipping for now
  • Backports requested where applicable (ex: critical bugfixes) => not needed?

@PVince81
Copy link
Member Author

/backport to stable25

@PVince81
Copy link
Member Author

/backport to stable24

@PVince81
Copy link
Member Author

/backport to stable23

@nickvergessen
Copy link
Member

Can we retest for the SSO tests?

@PVince81
Copy link
Member Author

pffff those env issues... restarted a second time

Extend the external storage configuration parameters definition to allow
to specify a default value

Signed-off-by: Vincent Petry <[email protected]>
@PVince81 PVince81 force-pushed the enh/noid/ext-storage-default-values branch from 344a824 to 4e179e0 Compare January 13, 2023 09:27
@PVince81
Copy link
Member Author

fixed definition parameter unit test then squashed + rebased

The SMB SSO tests will likely not boot because the env is broken (user_saml). The reason we don't always see that failure is because those tests don't always run due to conditions based on files_external code.

@PVince81 PVince81 added the 4. to release Ready to be released and/or waiting for tests to finish label Jan 16, 2023
@PVince81
Copy link
Member Author

I've raised #36163 to look into the SSO tests.

I'll merge this now as SMB was not touched by this change.

@PVince81 PVince81 merged commit f0b9b6e into master Jan 16, 2023
@PVince81 PVince81 deleted the enh/noid/ext-storage-default-values branch January 16, 2023 13:22
@backportbot-nextcloud
Copy link

The backport to stable24 failed. Please do this backport manually.

@backportbot-nextcloud
Copy link

The backport to stable23 failed. Please do this backport manually.

@PVince81
Copy link
Member Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish enhancement feature: external storage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants