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

Update cmdlets for Entra integration #832

Closed
FH-Inway opened this issue Jun 8, 2024 · 1 comment · Fixed by #833
Closed

Update cmdlets for Entra integration #832

FH-Inway opened this issue Jun 8, 2024 · 1 comment · Fixed by #833
Assignees
Labels
enhancement New feature or request

Comments

@FH-Inway
Copy link
Member

FH-Inway commented Jun 8, 2024

The cmdlets New-D365EntraIntgration and Test-D365EntraIntegration were created based on the Microsoft documentation https://learn.microsoft.com/en-us/dynamics365/fin-ops-core/dev-itpro/dev-tools/secure-developer-vm#external-integrations.

The cmdlets were based on version 9b7a2511 of the documentation. Since then, Microsoft updated the documentation. The cmdlets should be updated to reflect the changes made.

In detail, the changes were made to enable 2 additional capabilities:

  • Report execution using WCF service
  • Data Task Automation

Changes

1. Adding the Entra AppId in wif.config

This has already been added to the cmdlets with the optional -AddAppRegistrationToWifConfig switch. Since it is now officially recommended, the switch is to be removed and the adding is to always be done.

As a minor change, the entry is to be added after instead of the before the existing entry.

2. Clear cached LCS configuration in AxDB

This is to be added.

3. Restart Internet Information Services (IIS)

This is to be added.

FH-Inway added a commit that referenced this issue Jun 8, 2024
Based on the latest Microsoft recommendation for the Entra integration (https://learn.microsoft.com/en-us/dynamics365/fin-ops-core/dev-itpro/dev-tools/secure-developer-vm#external-integrations), the following cmdlets were updated:
- New-D365EntraIntegration
  - remove `-AddAppRegistrationToWifConfig` switch, this is now always done
  - clear cached LCS configuration in AxDB
  - restart IIS
- Test-D365EntraIntegration
  - A failing test for wif.config now results in an error

fixed #832
@FH-Inway FH-Inway self-assigned this Jun 8, 2024
@FH-Inway FH-Inway added the enhancement New feature or request label Jun 8, 2024
@FH-Inway
Copy link
Member Author

This is now available in version 0.7.13.

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

Successfully merging a pull request may close this issue.

1 participant