Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Smtp Deployment step #5556

Closed
wants to merge 4 commits into from
Closed

Smtp Deployment step #5556

wants to merge 4 commits into from

Conversation

agriffard
Copy link
Member

Fixes #5493

Import works but the password is encrypted based on the data protection key so it won't work on another tenant.

@deanmarcussen
Copy link
Member

You could remove the password field from the export, as is done with the Id property in the contents step

Then it's no longer part of the schema, so is more known that it isn't possible to export/import it.

objectData.Remove(nameof(ContentItem.Id));

@agriffard
Copy link
Member Author

Yes, but people will not understand that the password is not exported/imported.

@agriffard agriffard mentioned this pull request Feb 15, 2020
26 tasks
@sebastienros
Copy link
Member

I think we should have the encrypted settings (any of them) exported as a different step. This way you can decide if you want to export them as you might know that it will work or not (for instance sharing the protection keys between a staging and production, either on the FS, blob storage, database, ...).

Another option would be to decrypt them, send them unencrypted, then re-encrypt locally. This could be an option of the specific step for encrypted data, to decrypt it or not.

In the end we would have these options:

  • deploy without secrets
  • deploy the secrets encrypted using the custom secret step. The DP keys need to match on both ends, and using a File export is safe. Deploying a secret is opt-in.
  • deploy the secrets unencrypted (if the option of the setting step is set) and File export should be disabled, or only accessible with specific permissions. But the devs take the responsibility to do that.

@sebastienros
Copy link
Member

TODO to merge this PR: Remove the encrypted secrets from the payload.
On a separate PR we will create a custom step to export these encrypted values.
On another PR we will add an option to export these values unencrypted.

@JoshLefebvre
Copy link
Contributor

Any further progress on this PR? This is an important one for me as I would really like to automate the process of setting up Smtp settings for new tenants

@agriffard
Copy link
Member Author

Replaced by #6421

@agriffard agriffard closed this Jun 23, 2020
@agriffard agriffard deleted the ag/smtpDeploymentStep branch October 5, 2021 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Export/Import Smtp Settings and decrypt Global method
4 participants