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

Generated export command excludes resources #1759

Closed
ykuijs opened this issue Feb 11, 2022 · 2 comments · Fixed by #2230
Closed

Generated export command excludes resources #1759

ykuijs opened this issue Feb 11, 2022 · 2 comments · Fixed by #2230
Assignees
Labels
Core Engine Enhancement New feature or request

Comments

@ykuijs
Copy link
Member

ykuijs commented Feb 11, 2022

When I generate an export on the export.microsoft365dsc.com website, select all resources and select Application as Authentication method, running the command gives me this prompt:
image

Wouldn't it make sense to disable all resources that don't support the selected authentication method on the Export page?

@andikrueger
Copy link
Collaborator

This would make a lot of sense. We would need this information somewhere for the website. Maybe we could add this information to the settings files too (see #1751)

@ykuijs
Copy link
Member Author

ykuijs commented Jul 6, 2022

I am working on an update of the Export functionality that will add support for specifying multiple authentication method. When you do, the code will use the most secure method that is the greatest denominator between what is provided on the command line and what is supported by the resource. It is using the following order (from most secure to less secure): Certificate Thumbprint, Certificate Path, Application Secret, Credential

For example:

  • The resource supports Credential and CertificatePath
  • For the Export-M365DSCConfiguration Credential, CertificatePath and CertificateThumbprint are specified.

Since the resource does not support CertificateThumbprint, the code is then using CertificatePath to export the resource.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core Engine Enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants