Skip to content

AADInstance null in CSP and CPV examples #55

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

Open
dkattan opened this issue Aug 24, 2020 · 0 comments
Open

AADInstance null in CSP and CPV examples #55

dkattan opened this issue Aug 24, 2020 · 0 comments

Comments

@dkattan
Copy link

dkattan commented Aug 24, 2020

Overview

Samples don't work out of the box even when populating App.config with valid parameters

Reproduction

Create App Registration in a Patrner tenant using the following script:
https://gcits.com/knowledge-base/how-to-connect-to-delegated-office-365-tenants-using-the-secure-app-model/

Clone repo, plugin parameters returned from script into App.config

Expected Behavior

The application should not throw an except

Actual Behavior

Exception is thrown because AADInstance variable is null

Possible solutions

Change the following lines from

private static readonly string AADInstance = ConfigurationManager.AppSettings["AADInstance"];

to
private static readonly string AADInstance = ConfigurationManager.AppSettings["ida:AADInstance"];

private static readonly string AADInstance = ConfigurationManager.AppSettings["AADInstance"];

private static readonly string AADInstance = ConfigurationManager.AppSettings["AADInstance"];

@dkattan dkattan changed the title AADInstance null in CSP and CPV exampls AADInstance null in CSP and CPV examples Aug 24, 2020
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

No branches or pull requests

1 participant